On Tue, Jun 28, 2022 2:18 PM vignesh C <vignes...@gmail.com> wrote: > > Thanks for the comments, the attached v25 patch has the changes for the > same. >
Thanks for updating the patch. Here are some comments. 0002 patch: ============== 1. +# Test the CREATE SUBSCRIPTION 'origin' parameter and its interaction with +# 'copy_data' parameter. It seems we should move "and its interaction with 'copy_data' parameter" to 0003 patch. 0003 patch ============== 1. When using ALTER SUBSCRIPTION ... REFRESH, subscription will throw an error if any table is subscribed in publisher, even if the table has been subscribed before refresh (which won't do the initial copy when refreshing). It looks the previously subscribed tables don't need this check. Would it be better that we only check the tables which need to do the initial copy? 2. + errmsg("table:%s.%s might have replicated data in the publisher", + nspname, relname), I think the table name needs to be enclosed in double quotes, which is consistent with other messages. Regards, Shi yu