Hi, Patch v22-0001 LGTM apart from the following nitpicks
======
src/sgml/ref/alter_subscription.sgml
nitpick - /one needs to/you need to/
======
src/backend/commands/subscriptioncmds.c
CheckAlterSubOption:
nitpick = "ideally we could have..." doesn't make sense because the
code uses a more consistent/simpler way. So other option was not ideal
after all.
AlterSubscription
nitpick - typo /syncronization/synchronization/
nipick - plural fix
======
Kind Regards,
Peter Smith.
Fujitsu Australia.
