RE: pg_createsubscriber: drop pre-existing subscriptions from the converted node
Hayato Kuroda (Fujitsu) <kuroda.hayato@fujitsu.com>
From: "Hayato Kuroda (Fujitsu)" <kuroda.hayato@fujitsu.com>
To: 'Amit Kapila' <amit.kapila16@gmail.com>, vignesh C <vignesh21@gmail.com>
Cc: "pgsql-hackers@lists.postgresql.org" <pgsql-hackers@lists.postgresql.org>, Euler Taveira <euler@eulerto.com>, Peter Eisentraut <peter.eisentraut@enterprisedb.com>
Date: 2024-06-27T06:17:18Z
Lists: pgsql-hackers
Attachments
- v2-0001-pg_createsubscriber-remove-unused-attribute.patch (application/octet-stream) patch v2-0001
- v2-0002-pg_createsubscriber-Drop-pre-existing-subscriptio.patch (application/octet-stream) patch v2-0002
Dear Amit, Vingesh, Thanks for giving comments! > It seems disabling subscriptions on the primary can make the primary > stop functioning for some duration of time. I feel we need some > solution where after converting to subscriber, we disable and drop > pre-existing subscriptions. One idea could be that we use the list of > new subscriptions created by the tool such that any subscription not > existing in that list will be dropped. Previously I avoided coding like yours, because there is a room that converted node can connect to another publisher. But per off-list discussion, we can skip it by setting max_logical_replication_workers = 0. I refactored with the approach. Note that the GUC is checked at verification phase, so an attribute is added to start_standby_server() to select the workload. Most of comments by Vignesh were invalidated due to the code change, but I hoped I checked your comments were not reproduced. Also, 0001 was created to remove an unused attribute. > Shouldn't this be an open item for PG17? Added this thread to wikipage. Best Regards, Hayato Kuroda FUJITSU LIMITED https://www.fujitsu.com/
Commits
-
Drop pre-existing subscriptions from the converted subscriber.
- 4867f8a555ce 18.0 landed
- 622cb84d69be 17.0 landed
-
Remove unused structure member in pg_createsubscriber.c.
- a4c87df43a45 18.0 landed
- ca522e2a896c 17.0 landed