Re: Single transaction in the tablesync worker?

Peter Smith <smithpb2250@gmail.com>

View thread View on postgr.es

From: Peter Smith <smithpb2250@gmail.com>

To: Amit Kapila <amit.kapila16@gmail.com>

Cc: PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>

Date: 2021-01-29T10:37:15Z

Lists: pgsql-hackers

Attachments

v22-0001-Tablesync-Solution1.patch (application/octet-stream) patch v22-0001
v22-0002-Tablesync-extra-logging.patch (application/octet-stream) patch v22-0002

Hi Amit.

PSA the v22 patch for the Tablesync Solution1.

Differences from v21:
+ Patch is rebased to latest OSS HEAD @ 29/Jan.
+ Includes new code as suggested [ak0128] to ensure no dangling slots
at Drop/AlterSubscription.
+ Removes the slot/origin cleanup down by process interrupt logic
(cleanup_at_shutdown function).
+ Addresses some minor review comments.

----
[ak0128] https://www.postgresql.org/message-id/CAA4eK1LMYXZY1SpzgW-WyFdy%2BFTMZ4BMz1dj0rT2rxGv-zLwFA%40mail.gmail.com

Kind Regards,
Peter Smith.
Fujitsu Australia

Commits

Fix relcache reference leak introduced by ce0fdbfe97.
- 8a4f9522d0c7 14.0 landed
Fix Subscription test added by commit ce0fdbfe97.
- c8b21b037814 14.0 landed
Allow multiple xacts during table sync in logical replication.
- ce0fdbfe9722 14.0 landed
Logical replication support for initial data copy
- 7c4f52409a8c 10.0 cited