| From: | shveta malik <shveta(dot)malik(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>, Peter Smith <smithpb2250(at)gmail(dot)com>, shveta malik <shveta(dot)malik(at)gmail(dot)com> |
| Subject: | Re: DOCS - Add missing EXCEPT parameter description to ALTER PUBLICATION |
| Date: | 2026-06-09 05:25:13 |
| Message-ID: | CAJpy0uBgieNR2prF-K-a33RQ21kTdDRri4pN89ziHd56Pn8ZMg@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Tue, Jun 9, 2026 at 10:52 AM shveta malik <shveta(dot)malik(at)gmail(dot)com> wrote:
>
> On Tue, Jun 9, 2026 at 10:12 AM Nisha Moond <nisha(dot)moond412(at)gmail(dot)com> wrote:
> >
> > Hi Hackers,
> >
> > As Peter noted in [1], the ALTER PUBLICATION documentation is missing
> > a parameter description for the EXCEPT clause. Ref: Commit fd36606
> > introduced the EXCEPT clause.
> >
> > Attached is a simple patch to document the clause and its behavior.
> >
> > [1] https://www.postgresql.org/message-id/CAHut%2BPs3sghX4qv1jehqMgvZG7DmUoAFqgjmVc5xUy%2Bv5kHN3w%40mail.gmail.com
> >
>
> The patch LGTM.
>
> I have one more point to discuss. The EXCEPT implementation also
> introduced a new ALTER-PUB variation where users can convert an ALL
> TABLE to ALL SEQ and vice versa. We have not explained that anywhere.
> I am not sure if a one-line explanation can be added. If not, I think
> we can at-least add an example at the end for the same. Thoughts?
>
> The existing examples are:
>
> ~~
> Replace the table list in the publication's EXCEPT clause:
> ALTER PUBLICATION mypublication SET ALL TABLES EXCEPT (TABLE users,
> departments);
>
> Reset the publication to be a FOR ALL TABLES publication with no
> excluded tables:
> ALTER PUBLICATION mypublication SET ALL TABLES;
> ~~
>
> We can add one more:
> Convert an ALL TABLES publication to ALL SEQUENCES one:
> ALTER PUBLICATION mypublication SET ALL SEQUENCES;
>
Okay, I see we already have this line in doc in initial explanation:
The SET ALL TABLES clause can transform an empty publication, or one
defined for ALL SEQUENCES (or both ALL TABLES and ALL SEQUENCES), into
a publication defined for ALL TABLES. Likewise, SET ALL SEQUENCES can
convert an empty publication, or one defined for ALL TABLES (or both
ALL TABLES and ALL SEQUENCES), into a publication defined for ALL
SEQUENCES.
~~
So please ignore my comment about a one-line explanation. An example
may be added though (if others also think so).
thanks
Shveta
| From | Date | Subject | |
|---|---|---|---|
| Next Message | shaobo zhang | 2026-06-09 05:41:31 | Re: Fix missing semicolon in pl_gram.y for option_value rule |
| Previous Message | shveta malik | 2026-06-09 05:22:50 | Re: DOCS - Add missing EXCEPT parameter description to ALTER PUBLICATION |