On Wed, Jan 26, 2022 at 6:28 AM Michael Paquier <mich...@paquier.xyz> wrote: > > On Tue, Jan 25, 2022 at 07:20:05PM +0000, Bossart, Nathan wrote: > > I looked into removing the "shutdown" variable in favor of using > > "flags" everywhere, but the patch was quite messy and repetitive. I > > think another way to make things less confusing is to replace > > "shutdown" with an inverse variable called "online." The attached > > patch does it this way. > > Yeah, that sounds like a good compromise. At least, I find the whole > a bit easier to follow.
v3 LGTM. > Heikki was planning to commit a large refactoring of xlog.c, so we'd > better wait for that to happen before concluding here. Will that patch set fix the issue reported here? Regards, Bharath Rupireddy.
