pgsql: Change syntax of EXCEPT TABLE clause in publication commands. - Mailing list pgsql-committers

From Amit Kapila
Subject pgsql: Change syntax of EXCEPT TABLE clause in publication commands.
Date
Msg-id E1w7QYy-002FWz-1t@gemulon.postgresql.org
Whole thread Raw
Responses Re: pgsql: Change syntax of EXCEPT TABLE clause in publication commands.
List pgsql-committers
Change syntax of EXCEPT TABLE clause in publication commands.

Adjust the syntax of the EXCEPT clause in CREATE/ALTER PUBLICATION
added in commits fd366065e0 and 493f8c6439 to move the TABLE keyword
inside the relation list.

Old syntax:
CREATE PUBLICATION ... FOR ALL TABLES EXCEPT TABLE (t1, ...);
ALTER PUBLICATION  ... SET ALL TABLES EXCEPT TABLE (t1, ...);

New syntax:
CREATE PUBLICATION ... FOR ALL TABLES EXCEPT (TABLE t1, ...);
ALTER PUBLICATION  ... SET ALL TABLES EXCEPT (TABLE t1, ...);

This is to ensure that inclusion and exclusion list can be specified in
a same way. Previously, the exclusion table list can be specified as
TABLE (t1, t2, t3) and inclusion list can be specified as TABLE t1, t2,
t3, or TABLE t1, TABLE t2, TABLE t3.

This change is purely syntactic and does not alter behavior.

Reported-by: Masahiko Sawada <sawada.mshk@gmail.com>
Author: vignesh C <vignesh21@gmail.com>
Author: Shlok Kyal <shlok.kyal.oss@gmail.com>
Reviewed-by: shveta malik <shveta.malik@gmail.com>
Reviewed-by: Peter Smith <smithpb2250@gmail.com>
Reviewed-by: Zhijie Hou <houzj.fnst@fujitsu.com>
Reviewed-by: Dilip Kumar <dilipbalaut@gmail.com>
Reviewed-by: SATYANARAYANA NARLAPURAM <satyanarlapuram@gmail.com>
Reviewed-by: Amit Kapila <amit.kapila16@gmail.com>
Discussion: https://postgr.es/m/CAD21AoCC8XuwfX62qKBSfHUAoww_XB3_84HjswgL9jxQy696yw@mail.gmail.com
Discussion: https://postgr.es/m/CALDaNm3=JrucjhiiwsYQw5-PGtBHFONa6F7hhWCXMsGvh=tamA@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/5984ea868eeeb202bddedfbdaca81adfe7ea6779

Modified Files
--------------
doc/src/sgml/catalogs.sgml                |  2 +-
doc/src/sgml/logical-replication.sgml     |  2 +-
doc/src/sgml/ref/alter_publication.sgml   | 28 ++++++------
doc/src/sgml/ref/create_publication.sgml  | 20 +++++----
src/backend/catalog/pg_publication.c      |  6 +--
src/backend/commands/publicationcmds.c    |  2 +-
src/backend/parser/gram.y                 | 10 ++---
src/bin/pg_dump/pg_dump.c                 | 16 +++----
src/bin/pg_dump/t/002_pg_dump.pl          | 12 +++---
src/bin/psql/describe.c                   |  2 +-
src/bin/psql/tab-complete.in.c            | 28 ++++++------
src/test/regress/expected/publication.out | 71 ++++++++++++++++++++-----------
src/test/regress/sql/publication.sql      | 59 ++++++++++++++-----------
src/test/subscription/t/037_except.pl     | 52 +++++++++++-----------
14 files changed, 176 insertions(+), 134 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Doc: update ddl.sgml's description of cmin and cmax.
Next
From: Pavel Stehule
Date:
Subject: Re: pgsql: Change syntax of EXCEPT TABLE clause in publication commands.