On Fri, May 30, 2025 at 08:32:05AM +0900, Michael Paquier wrote: > On Thu, May 29, 2025 at 08:28:01PM +1000, Ajin Cherian wrote: >> I think this new fix is much better and cleaner. A suggestion for the >> comment: >> "For cascading logical WAL senders, we use the replay LSN instead of the >> flush LSN, since logical decoding on a standby only processes WAL that has >> been replayed. This distinction becomes particularly important during >> shutdown, as new WAL is no longer replayed and the last replayed LSN marks >> the furthest point up to which decoding can proceed." > > Yes, that sounds better than my previous suggestion. Thanks.
I have reused that wording, did more tests on v16 and v17, reproducing both the problems reported and that the change fixes the shutdown of the physical standby, then applied the change down to v16.