On Mon, Nov 8, 2021 at 10:31 PM Michael Paquier <mich...@paquier.xyz> wrote: > I think that this patch is an improvement.
Cool. > @@ -6686,8 +6682,8 @@ StartupXLOG(void) > - TimeLineID ThisTimeLineID, > - PrevTimeLineID; > + TimeLineID replayTLI, > + newTLI; > One problem with newTLI is that this conflicts with the declaration > around 7663 in xlog.c, where we check after a TLI switch when > replaying such a record. Perhaps this could be named newInsertTLI, > for example. That's a good point. However, since I think newTLI is already in use in some of the functions StartupXLOG() calls, and since I think it would be good to use the same name in StartupXLOG() as in the functions that it calls, what I'd prefer to do is rename the newTLI variable in the inner scope to newReplayTLI, as in the attached v2. -- Robert Haas EDB: http://www.enterprisedb.com
v2-0001-More-cleanup-of-ThisTimeLineID.patch
Description: Binary data
