Hi Nisha. No further comments for patches v15-0001,0002,0003.
But, here are some comments for v15-0004. ====== doc/src/sgml/ref/alter_publication.sgml 1. + <para> + For <literal>FOR TABLES IN SCHEMA</literal> publications, the + <literal>EXCEPT</literal> clause is schema-scoped. If a table listed in + the <literal>EXCEPT</literal> clause is later moved to a different schema + using <command>ALTER TABLE ... SET SCHEMA</command>, the exclusion is + removed; the table will then be published if its new schema is part of a + publication. If the table is subsequently moved back to the original + schema, the exclusion is not restored, and must be re-established + explicitly using <command>ALTER PUBLICATION</command>. + </para> IIRC the recent commit 77b6dd9 that explained about moving tables to other schemas, was only on the CREATE PUBLICATION page; not on the ALTER PUBLICATION page. So either: (a) maybe this entire paragraph is not needed, or (b) keep it, but then the behaviour for FOR ALL TABLES also needs to be described here ====== doc/src/sgml/ref/alter_publication.sgml 2. Once a table is excluded, the exclusion applies to that table regardless of its name or schema. Renaming the table or moving it to another schema using ALTER TABLE ... SET SCHEMA does not remove the exclusion. ~~ The recent commit 77b6dd9 added the above text to the CREATE PUBLICATION page. But, that was correct only for "FOR ALL TABLES" -- after your patch we can't say that unconditionally anymore. Also, since that is mentioned on the CREATE PUBLICATION page, I guess we need to say something similar about ALL TABLES IN SCHEMA ... EXCEPT in the same paragraph. So, maybe something like below: SUGGESTION Once a table is excluded under <literal>FOR ALL TABLES</literal>, the exclusion applies to that table, the exclusion applies to that table regardless of its name or schema. Renaming the table or moving it to another schema using <command>ALTER TABLE ... SET SCHEMA</command> does not remove the exclusion. However, for <literal>FOR TABLES IN SCHEMA</literal>, because the <literal>EXCEPT</literal> is schema-scoped, moving a schema-excluded table to another schema does remove the exclusion. ====== Kind Regards, Peter Smith. Fujitsu Australia
