pgsql: Fix ALTER SUBSCRIPTION ... SET PUBLICATION ... command. - Mailing list pgsql-committers

From Amit Kapila
Subject pgsql: Fix ALTER SUBSCRIPTION ... SET PUBLICATION ... command.
Date
Msg-id E1tsw0F-002NUB-2s@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix ALTER SUBSCRIPTION ... SET PUBLICATION ... command.

The problem is that ALTER SUBSCRIPTION ... SET PUBLICATION ... will lead
to restarting of apply worker and after the restart, the apply worker will
use the existing slot and replication origin corresponding to the
subscription. Now, it is possible that before the restart, the origin has
not been updated, and the WAL start location points to a location before
where PUBLICATION pointed to by SET PUBLICATION doesn't exist, and that
can lead to an error like: "ERROR:  publication "pub1" does not exist".
Once this error occurs, apply worker will never be able to proceed and
will always return the same error.

We decided to skip loading the publication if the publication does not
exist. The publication is loaded later and updates the relation entry when
the publication gets created.

We decided not to backpatch this as this is a behaviour change, and we don't
see field reports. This problem has been found by intermittent buildfarm
failures.

Author: vignesh C <vignesh21@gmail.com>
Reviewed-by: Dilip Kumar <dilipbalaut@gmail.com>
Reviewed-by: Amit Kapila <amit.kapila16@gmail.com>
Reviewed-by: Masahiko Sawada <sawada.mshk@gmail.com>
Discussion: https://postgr.es/m/flat/CALDaNm0-n8FGAorM%2BbTxkzn%2BAOUyx5%3DL_XmnvOP6T24%2B-NcBKg%40mail.gmail.com
Discussion: https://postgr.es/m/CAA4eK1+T-ETXeRM4DHWzGxBpKafLCp__5bPA_QZfFQp7-0wj4Q@mail.gmail.com

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/7c99dc587a010a0c40d72a0e435111ca7a371c02

Modified Files
--------------
src/backend/replication/pgoutput/pgoutput.c | 16 +++++++++--
src/test/subscription/t/024_add_drop_pub.pl | 44 ++++++++++++++++++++++++++++-
2 files changed, 57 insertions(+), 3 deletions(-)


pgsql-committers by date:

Previous
From: Tom Lane
Date:
Subject: pgsql: Fix ARRAY_SUBLINK and ARRAY[] for int2vector and oidvector input
Next
From: Thomas Munro
Date:
Subject: pgsql: Improve buffer manager API for backend pin limits.