On Sun, Jun 7, 2020 at 10:49 PM Thomas Munro <thomas.mu...@gmail.com> wrote: > > On Wed, Jun 3, 2020 at 2:03 PM Michael Paquier <mich...@paquier.xyz> wrote: > > On Wed, Jun 03, 2020 at 10:56:13AM +1200, Thomas Munro wrote: > > > Sorry for my radio silence, I got tangled up with a couple of > > > conferences. I'm planning to look at 0001 and 0002 shortly. > > > > Thanks! > > I pushed 0001 and 0002, squashed into one commit. I'm not sure about > 0003. If we're going to do that, wouldn't it be better to just > acquire the lock in that one extra place in StartupXLOG(), rather than > introducing the extra parameter?
Today, after committing a3e6c6f, I saw recovery/018_wal_optimize.pl fail and see this message in the replica log [2]. 2024-05-16 15:12:22.821 GMT [5440][not initialized] FATAL: incorrect checksum in control file I'm pretty sure it's not related to my commit. So, I was looking for existing reports of this error message. It's a long shot, since 0001 and 0002 were already pushed, but this is the only recent report I could find of "FATAL: incorrect checksum in control file" in pgsql-hackers or bugs archives. I do see this thread from 2016 [3] which might be relevant because the reported bug was also on Windows. - Melanie [1] https://cirrus-ci.com/task/4626725689098240 [2] https://api.cirrus-ci.com/v1/artifact/task/4626725689098240/testrun/build/testrun/recovery/018_wal_optimize/log/018_wal_optimize_node_replica.log [3] https://www.postgresql.org/message-id/flat/CAEepm%3D0hh_Dvd2Q%2BfcjYpkVzSoNX2%2Bf167cYu5nwu%3Dqh5HZhJw%40mail.gmail.com#042e9ec55c782370ab49c3a4ef254f4a