Improve ALTER PUBLICATION validation and error messages
Attempting to add a system column for a table to an existing publication
would result in the not very intuitive error message of:
ERROR: negative bitmapset member not allowed
Here we improve that to have it display the same error message as a user
would see if they tried adding a system column for a table when adding
it to the publication in the first place.
Doing this requires making the function which validates the list of
columns an extern function. The signature of the static function wasn't
an ideal external API as it made the code more complex than it needed to be.
Here we adjust the function to have it populate a Bitmapset of attribute
numbers. Doing it this way allows code simplification.
There was no particular bug here other than the weird error message, so
no backpatch.
Bug: #18558
Reported-by: Alexander Lakhin <exclusion@gmail.com>
Author: Peter Smith, David Rowley
Discussion: https://postgr.es/m/18558-411bc81b03592125@postgresql.org
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/80ffcb842748f0b8ccf8b24c5c3b2500755d2dcd
Modified Files
--------------
src/backend/catalog/pg_publication.c | 110 +++++++++++++-----------------
src/backend/commands/publicationcmds.c | 22 ++----
src/backend/commands/subscriptioncmds.c | 2 +-
src/include/catalog/pg_publication.h | 1 +
src/test/regress/expected/publication.out | 7 ++
src/test/regress/sql/publication.sql | 4 ++
6 files changed, 66 insertions(+), 80 deletions(-)
