On Mon, 16 Jan 2023 at 04:49, Tomas Vondra <tomas.von...@enterprisedb.com> wrote: > > cfbot didn't like the rebased / split patch, and after looking at it I > believe it's a bug in parallel apply of large transactions (216a784829), > which seems to have changed interpretation of in_remote_transaction and > in_streamed_transaction. I've reported the issue on that thread [1], but > here's a version with a temporary workaround so that we can continue > reviewing it. >
The patch does not apply on top of HEAD as in [1], please post a rebased patch: === Applying patches on top of PostgreSQL commit ID 17e72ec45d313b98bd90b95bc71b4cc77c2c89c3 === === applying patch ./0001-Fix-snapshot-handling-in-logicalmsg_decode-20230116.patch patching file src/backend/replication/logical/decode.c patching file src/backend/replication/logical/reorderbuffer.c === applying patch ./0002-Logical-decoding-of-sequences-20230116.patch patching file doc/src/sgml/logicaldecoding.sgml Hunk #3 FAILED at 483. Hunk #4 FAILED at 494. Hunk #7 succeeded at 1252 (offset 4 lines). 2 out of 7 hunks FAILED -- saving rejects to file doc/src/sgml/logicaldecoding.sgml.rej [1] - http://cfbot.cputube.org/patch_41_3823.log Regards, Vignesh