Re: Data is copied twice when specifying both child and parent table in publication
Amit Kapila <amit.kapila16@gmail.com>
From: Amit Kapila <amit.kapila16@gmail.com>
To: Jacob Champion <jchampion@timescale.com>
Cc: "wangw.fnst@fujitsu.com" <wangw.fnst@fujitsu.com>,
Andres Freund <andres@anarazel.de>, vignesh C <vignesh21@gmail.com>, Peter Smith <smithpb2250@gmail.com>, "Takamichi Osumi (Fujitsu)" <osumi.takamichi@fujitsu.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>
Date: 2023-03-21T06:21:51Z
Lists: pgsql-hackers
On Mon, Mar 20, 2023 at 11:22 PM Jacob Champion <jchampion@timescale.com> wrote: > > On Fri, Mar 17, 2023 at 9:45 PM Amit Kapila <amit.kapila16@gmail.com> wrote: > > > There are a bunch of moving parts and hidden subtleties here, and I fell > > > into a few traps when I was working on my patch, so it'd be nice to have > > > additional coverage. I'm happy to contribute effort in that area if it's > > > helpful. > > > > I think it depends on what tests you have in mind. > > Just the ones I mentioned, to start with. > > > I suggest you can > > propose a patch to cover tests for this are in a separate thread. We > > can then evaluate those separately. > > To confirm -- you want me to start a new thread for tests for this > patchset? (Tests written against HEAD would likely be obsoleted by > this change.) > If the tests you have in mind are only related to this patch set then feel free to propose them here if you feel the current ones are not sufficient. I just want to be cautious that we shouldn't spend too much time adding additional tests which are related to the base functionality as we have left with less time for the last CF and I would like to push the change for HEAD before that. -- With Regards, Amit Kapila.
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