On Mon, Jan 10, 2022 at 02:31:38PM -0500, Tom Lane wrote: > > So I think we need to fix it to capture the target WAL position > at the start, as I've done in the attached patch.
+1, it looks sensible to me. > In principle > this might make things a bit slower because of the extra > transaction required, but I don't notice any above-the-noise > difference on my own workstation. I'm wondering if the environments where this extra transaction could make a noticeable difference are also environments where doing that extra transaction can save some iteration(s), which would be at least as costly.