Re: Separate HEAP WAL replay logic into its own file - Mailing list pgsql-hackers

From Li, Yong
Subject Re: Separate HEAP WAL replay logic into its own file
Date
Msg-id CC5477BD-59AF-4889-AA32-E77ABCBB1971@ebay.com
Whole thread Raw
In response to Re: Separate HEAP WAL replay logic into its own file  (Melanie Plageman <melanieplageman@gmail.com>)
List pgsql-hackers

> On Jun 18, 2024, at 20:42, Melanie Plageman <melanieplageman@gmail.com> wrote:
> 
> External Email
> 
> On Mon, Jun 17, 2024 at 9:12 PM Li, Yong <yoli@ebay.com> wrote:
>> 
>> As a newcomer, when I was walking through the code looking for WAL replay related code, it was relatively easy for
meto find them for the B-Tree access method because of the “xlog” hint in the file names. It took me a while to find
thesame for the heap access method. When I finally found them (via text search), it was a small surprise. Having
differentfile organizations for different access methods gives me this urge to make everything consistent. I think it
willmake it easier for newcomers, and it will reduce the mental load for everyone to remember that heap replay is
insidethe heapam.c not some “???xlog.c”.
 
> 
> That makes sense. The branch for PG18 has not been cut yet, so I
> recommend registering this patch for the July commitfest [1] so it
> doesn't get lost.
> 
> - Melanie
> 

Thanks for the positive feedback. I’ve added the patch to the July CF.

Yong


pgsql-hackers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: Conflict Detection and Resolution
Next
From: Amit Langote
Date:
Subject: Re: Incorrect matching of sql/json PASSING variable names