Re: Improve CREATE/ALTER PUBLICATION syntax for EXCEPT

Some review comments for v2.

1. (rendered from patch)
Likewise altering a publication to set ALL TABLES, to change the
EXCEPT list, or to set or unset ALL SEQUENCES also requires the
subscriber to refresh the publication.

~

The text there says "set or unset" ALL SEQUENCES, but it only says
"set" ALL TABLES.

AFAIK the SET will *replace* the kind of publication.

e.g.1
Assuming we start out a publication that has FOR ALL SEQUENCES
Then, ALTER ... SET ALL TABLES will effectively "unset" ALL SEQUENCES

e.g.2
Assuming we start out a publication that has FOR ALL TABLES
Then, ALTER ... SET ALL SEQUENCES will effectively "unset" ALL TABLES

~

It might be better not to say "unset" at all, but if you want to keep
it then maybe it ought to be said in both places.

~~~

2.
(probably out of scope for this patch)

IMO this "Description" is becoming overly-complicated by saying "Note
that" and "Note also that" etc.

It seems to me that anything that is a note belongs in the "Notes"
section (but the "Notes" section is strangely missing from this page)
so that the "Description" section can be much less cluttered. e.g.
Compare with Description for CREATE PUBLICATION page.

Anyway, perhaps refactoring to make a new "Notes" section is a patch
for another day, not now.

======
Kind Regards,
Peter Smith.
Fujitsu Australia