RE: Data is copied twice when specifying both child and parent table in publication
Takamichi Osumi (Fujitsu) <osumi.takamichi@fujitsu.com>
From: "Takamichi Osumi (Fujitsu)" <osumi.takamichi@fujitsu.com>
To: "wangw.fnst@fujitsu.com" <wangw.fnst@fujitsu.com>, Peter Smith <smithpb2250@gmail.com>
Cc: Amit Kapila <amit.kapila16@gmail.com>, "shiy.fnst@fujitsu.com" <shiy.fnst@fujitsu.com>, "houzj.fnst@fujitsu.com" <houzj.fnst@fujitsu.com>, Amit Langote <amitlangote09@gmail.com>, Peter Eisentraut <peter.eisentraut@enterprisedb.com>, "pgsql-hackers@lists.postgresql.org" <pgsql-hackers@lists.postgresql.org>, Dilip Kumar <dilipbalaut@gmail.com>, Greg Nancarrow <gregn4422@gmail.com>, vignesh C <vignesh21@gmail.com>
Date: 2022-11-08T04:12:00Z
Lists: pgsql-hackers
On Monday, October 17, 2022 2:49 PM Wang, Wei/王 威 <wangw.fnst@fujitsu.com> wrote: > Attach the new patch set. Hi, thank you for posting the new patches. Here are minor comments on the HEAD_v13-0002. (1) Suggestion for the document description + <para> + If a root partitioned table is published by any subscribed publications which + set publish_via_partition_root = true, changes on this root partitioned table + (or on its partitions) will be published using the identity and schema of this + root partitioned table rather than that of the individual partitions. + </para> + I suppose this sentence looks quite similar to the one in the previous paragraph and can be adjusted. IIUC the main value of the patch is to clarify what happens when we mix publications of different publish_via_partition_root settings for one partition hierarchy. If this is true, how about below sentence instead of the one above ? " There can be a case where a subscription combines publications with different publish_via_partition_root values for one same partition hierarchy (e.g. subscribe two publications indicating the root partitioned table and its child table respectively). In this case, the identity and schema of the root partitioned table take priority. " (2) Better documentation alignment I think we need to wrap publish_via_partition_root by "literal" tag in the documentation create_publication.sgml. Best Regards, Takamichi Osumi
Commits
-
Avoid syncing data twice for the 'publish_via_partition_root' option.
- 062a84442424 16.0 landed
-
Fix partition table's REPLICA IDENTITY checking on the subscriber.
- 52d5ea9adb01 14.5 cited
-
Prohibit combining publications with different column lists.
- fd0b9dcebda7 15.0 cited
-
Fix double publish of child table's data.
- 3f06c00cf6dc 13.6 landed
- 614b77d65a38 14.2 landed
- 5e97905a2c76 15.0 landed
-
Support adding partitioned tables to publication
- 17b9e7f9fe23 13.0 cited