[COMMITTERS] pgsql: Fix DROP SUBSCRIPTION hang - Mailing list pgsql-committers

From Peter Eisentraut
Subject [COMMITTERS] pgsql: Fix DROP SUBSCRIPTION hang
Date
Msg-id E1dtlsH-0003h7-DF@gemulon.postgresql.org
Whole thread Raw
List pgsql-committers
Fix DROP SUBSCRIPTION hang

When ALTER SUBSCRIPTION DISABLE is run in the same transaction before
DROP SUBSCRIPTION, the latter will hang because workers will still be
running, not having seen the DISABLE committed, and DROP SUBSCRIPTION
will wait until the workers have vacated the replication origin slots.

Previously, DROP SUBSCRIPTION killed the logical replication workers
immediately only if it was going to drop the replication slot, otherwise
it scheduled the worker killing for the end of the transaction, as a
result of 7e174fa793a2df89fe03d002a5087ef67abcdde8.  This, however,
causes the present problem.  To fix, kill the workers immediately in all
cases.  This covers all cases: A subscription that doesn't have a
replication slot must be disabled.  It was either disabled in the same
transaction, or it was already disabled before the current transaction,
but then there shouldn't be any workers left and this won't make a
difference.

Reported-by: Arseny Sher <a.sher@postgrespro.ru>
Discussion: https://www.postgresql.org/message-id/flat/87mv6av84w.fsf%40ars-thinkpad

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/8edacab209957520423770851351ab4013cb0167

Modified Files
--------------
src/backend/commands/subscriptioncmds.c | 19 +++++++-----
src/test/subscription/t/007_ddl.pl      | 51 +++++++++++++++++++++++++++++++++
2 files changed, 63 insertions(+), 7 deletions(-)


--
Sent via pgsql-committers mailing list (pgsql-committers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-committers

pgsql-committers by date:

Previous
From: Michael Paquier
Date:
Subject: Re: [COMMITTERS] pgsql: Fix bogus size calculation introduced bycommit cc5f81366.
Next
From: Peter Eisentraut
Date:
Subject: [COMMITTERS] pgsql: Translation updates