| From: | Peter Smith <smithpb2250(at)gmail(dot)com> |
|---|---|
| To: | Nisha Moond <nisha(dot)moond412(at)gmail(dot)com> |
| Cc: | shveta malik <shveta(dot)malik(at)gmail(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Support EXCEPT for TABLES IN SCHEMA publications |
| Date: | 2026-04-29 06:32:12 |
| Message-ID: | CAHut+Pu5VNakf5JAhKM7T-P_q37eN1Qgv5nvZUe+8RAAT41y4g@mail.gmail.com |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Hi Nisha.
A couple of ad-hoc review comments for v4-0001...
======
src/bin/pg_dump/pg_dump.c
dumpPublicationNamespace:
1.
+ if (!first_except)
+ appendPQExpBufferStr(query, ")");
The logic seems ok, but the above seems a bit strange, checking
'first_except'. Maybe renaming to 'has_except' (and some small
refactoring) would read better?
~~~
2.
IIUC, the 'ALTER PUBLICATION' EXCEPT clause syntax change is not
introduced until your patch 0003. So, how does this dump code even
work when it relies on that ALTER PUBLICATION?
Furthermore, the patch 0003 commit message says 'ALTER PUBLICATION pub
SET TABLES', but this dump code is using 'ALTER PUBLICATION pub ADD
TABLES' (note ADD v SET). Something seems suspicious...
======
Kind Regards,
Peter Smith.
Fujitsu Australia
| From | Date | Subject | |
|---|---|---|---|
| Next Message | shveta malik | 2026-04-29 07:03:54 | Re: Proposal: Conflict log history table for Logical Replication |
| Previous Message | Dilip Kumar | 2026-04-29 06:20:15 | Re: Proposal: Conflict log history table for Logical Replication |