patch 002: Commit msg says: ---------- The command then re-fetches the publication tuple after acquiring the lock, but does not revalidate the publication relations. This can leave pg_publication_rel containing a mixture of inclusion and exclusion rows.
Re-run CheckAlterPublication() after acquiring the lock so that the validation uses the current publication state. --------- The commit message and the fix are not in sync and thus little confusing. Commit msg talks about re-validation while patch has shifted the validation to a later point. The code change looks good though. patch003: LGTM patch004: LGTM thanks Shveta
