Hi,
An update: I have two topics from the review perspective.
On the test I proposed to be added to 030_pg_recvlogical.pl, I found a
way to write it without using sleeps (which risk flakyness in CI). I've
attached it as a patch for your consideration. I checked the test in the
following way: on master it fails, but with your patch it passes.
Secondly I noticed in function sendFeedback at line 166, the startpos is
set to output_written_lsn. This seems to counter conceptually the change
you made in the patch, however it seems to not affect correctness. Shall
we remove this line as to avoid confusion?