Re: EquivalenceClass and custom_read_write

From: Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: EquivalenceClass and custom_read_write
Date: 2025-02-13 04:32:13
Message-ID: CAExHW5tcEQsATOBXaH5O0LuLK=1XKrJWeZOxH_Ju9aJEbfTC6Q@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Wed, Feb 12, 2025 at 10:25 PM Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> Ashutosh Bapat <ashutosh(dot)bapat(dot)oss(at)gmail(dot)com> writes:
> > In pathnodes.h
> > typedef struct EquivalenceClass
> > {
> > pg_node_attr(custom_read_write, no_copy_equal, no_read, no_query_jumble)
>
> > Because of custom_read_write attribute, I expect _outEquivalenceClass
> > to be present in outfuncs.c and _readEquivalenceClass to be present in
> > readfuncs.c. I find the first but not the second. This could be
> > because _out function is only for debugging and what it writes is
> > never read back. Should we add a comment in EquivalenceClass prologue
> > mentioning the same?
>
> Doesn't the "no_read" annotation imply exactly that?

Oh! I just looked at custom_read_write and missed no_read. Sorry.
Thanks for pointing that out.

--
Best Wishes,
Ashutosh Bapat

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2025-02-13 04:47:12 Re: Add isolation test template in injection_points for wait/wakeup/detach
Previous Message Thomas Munro 2025-02-13 04:15:58 Re: BitmapHeapScan streaming read user and prelim refactoring