On Mon, 29 Aug 2022 at 16:35, Amit Kapila <amit.kapil...@gmail.com> wrote: > > On Wed, Aug 24, 2022 at 7:27 PM vignesh C <vignes...@gmail.com> wrote: > > > > Since there was no objections to change it to throw a warning, I have > > made the changes for the same. > > > > Review comments for v42-0001* > ========================== > 1. Can we improve the query in check_pub_table_subscribed() so that it > doesn't fetch any table that is already part of the subscription on > the subscriber? This may be better than what the patch is doing which > is to first fetch such information and then skip it. If forming this > query turns out to be too complex then we can retain your method as > well but I feel it is worth trying to optimize the query used in the > patch.
I have modified the query to skip the tables as part of the query. The attached v45 patch has the changes for the same. Regards, Vignesh
v45-0001-Check-and-log-a-warning-if-the-publisher-has-sub.patch
Description: Binary data
v45-0002-Document-the-steps-for-replication-between-prima.patch
Description: Binary data
