pgsql: Improve ALTER PUBLICATION validation and error messages

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(at)gmail(dot)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(-)