On Fri, Jan 21, 2022 at 2:09 PM Masahiko Sawada <sawada.mshk@gmail.com> wrote:
>
> I've attached an updated patch that incorporated these commends as
> well as other comments I got so far.
>
src/backend/replication/logical/worker.c
(1)
Didn't you mean to say "check the" instead of "clear" in the following
comment? (the subtransaction's XID was never being cleared before,
just checked against the skipxid, and now that check has been removed)
+ * ... . Since we don't
+ * support skipping individual subtransactions we don't clear
+ * subtransaction's XID.
Other than that, the patch LGTM.
Regards,
Greg Nancarrow
Fujitsu Australia