| From: | Peter Smith <smithpb2250(at)gmail(dot)com> |
|---|---|
| To: | Nisha Moond <nisha(dot)moond412(at)gmail(dot)com> |
| Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
| Subject: | Re: DOCS - Add missing EXCEPT parameter description to ALTER PUBLICATION |
| Date: | 2026-06-11 00:57:50 |
| Message-ID: | CAHut+PudfQx8aGSi3xBtWJ-ATThcp4P_GJsQBRG+V3k2VgUvEg@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi Nisha,
Comments for patch v2-0001.
======
1.
In addition, SET ALL TABLES can be used to update the tables specified
in the EXCEPT clause of a FOR ALL TABLES publication. If EXCEPT is
specified with a list of tables, the existing exclusion list is
replaced with the specified tables. If EXCEPT is omitted, the existing
exclusion list is cleared.
~
The page already has the above test in the "Description".
I think one motivation of adding the "EXCEPT" part is to remove
clutter from the "Description", but currently there is a lot of
repetition. The "Description" can be pruned:
SUGGESTION
In addition, SET ALL TABLES can be used to update the tables specified
in the EXCEPT clause.
(and the EXCEPT above should link to the EXCEPT part)
======
Kind Regards,
Peter Smith.
Fujitsu Australia
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Michael Paquier | 2026-06-11 01:15:01 | Re: t/035_standby_logical_decoding.pl might fail on attempt to read wrong timeline |
| Previous Message | Jeff Davis | 2026-06-11 00:49:59 | Re: Commit Sequence Numbers and Visibility |