On Fri, Jun 20, 2025 at 5:48 AM Alexander Korotkov <aekorot...@gmail.com> wrote: > > > > > If what I said above is correct, then the following part of the commit > > message will be incorrect: > > "As stated in the ReplicationSlotReserveWal() comment, this is not > > always true. Additionally, this issue has been spotted by some > > buildfarm > > members." > > I agree, this comment needs improvement in terms of clarity. > > Meanwhile I've pushed the patch for TAP tests, which I think didn't > get any objections. >
Sounds reasonable. As per analysis till now, it seems removal of new assert is correct and we just need to figure out the reason in all failure cases as to why the physical slot's restart_lsn goes backward, and then add a comment somewhere to ensure that we don't repeat a similar mistake in the future. -- With Regards, Amit Kapila.
