Re: Obsolete comments in ResultRelInfo struct - Mailing list pgsql-hackers

From Etsuro Fujita
Subject Re: Obsolete comments in ResultRelInfo struct
Date
Msg-id CAPmGK16LJB7RLV_QBxN=XqR8HVXsp9gfDn_Y-Uw7Xo119Q4Qng@mail.gmail.com
Whole thread Raw
In response to Re: Obsolete comments in ResultRelInfo struct  (Etsuro Fujita <etsuro.fujita@gmail.com>)
List pgsql-hackers
On Tue, Aug 12, 2025 at 7:21 PM Etsuro Fujita <etsuro.fujita@gmail.com> wrote:
> On Tue, Aug 12, 2025 at 2:03 AM Dean Rasheed <dean.a.rasheed@gmail.com> wrote:
> > On Mon, 11 Aug 2025 at 12:25, Etsuro Fujita <etsuro.fujita@gmail.com> wrote:
> > >
> > >     /*
> > >      * Other information needed by child result relations
> > >      *
> > >      * RootResultRelInfo gives the target relation mentioned in the query.
> > >      * Used as the root for tuple routing and/or transition capture.
> > >      *
> > >      * PartitionTupleSlot is non-NULL if the relation is a partition to route
> > >      * tuples into and RootToChild conversion is needed.
> > >      */
> >
> > That seems reasonable.
>
> Cool!
>
> > I think it's also worth adding the "ri_" prefix
> > to the field names in those comments.
>
> +1, so I created a patch incorporating your proposal, which I am attaching.

Pushed and backpatched down to v14.

Best regards,
Etsuro Fujita



pgsql-hackers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: Compilation issues for HASH_STATISTICS and HASH_DEBUG options
Next
From: Tom Lane
Date:
Subject: Re: VM corruption on standby