On Tue, Nov 3, 2020 at 7:19 AM Dave Cramer <davecra...@gmail.com> wrote:
> David, > > On Thu, 24 Sep 2020 at 00:22, Michael Paquier <mich...@paquier.xyz> wrote: > >> On Tue, Sep 08, 2020 at 12:18:23PM -0700, Andres Freund wrote: >> > A test verifying that a non-transactional message in later aborted >> > transaction is handled correctly would be good. >> >> On top of that, the patch needs a rebase as it visibly fails to apply, >> per the CF bot. >> -- >> Michael >> > > Where are you with this? Are you able to work on it ? > Dave Cramer > Apologies for the delay, here. I've attached v2 of this patch which applies cleanly to master. The patch also now includes a test demonstrating that pg_logical_emit_message correctly sends non-transactional messages when called inside a transaction that is rolled back. (Thank you, Andres, for this suggestion.) The only other change is that I added this new message type into the LogicalRepMsgType enum added earlier this week. Let me know what you think. Cheers, Dave
v2-0001-Add-logical-decoding-messages-to-pgoutput.patch.gz
Description: application/gzip
