database/postgres
database/postgres
1
0
Fork 0
mirror of https://github.com/postgres/postgres.git synced 2025-05-01 01:04:50 +03:00
Code

Fix the errhint message and docs for drop subscription failure.

Browse Source 
The existing errhint message and docs were missing the fact that we can't
disassociate from the slot unless the subscription is disabled.

Author: Robert Sjöblom, Peter Smith
Reviewed-by: Peter Eisentraut, Amit Kapila
Backpatch-through: 11
Discussion: https://postgr.es/m/807bdf85-61ea-88e2-5712-6d9fcd4eabff@fortnox.se
This commit is contained in:
Amit Kapila 2023-06-21 10:36:09 +05:30
parent 5b1a879943
commit a734caa25f
2 changed files with 7 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
doc/src/sgml/ref/drop_subscription.sgml
View File

@ -85,9 +85,11 @@ DROP SUBSCRIPTION [ IF EXISTS ] <replaceable class="parameter">name</replaceable
for the subscription on the remote host are released. If this fails, for the subscription on the remote host are released. If this fails,
either because the remote host is not reachable or because the remote either because the remote host is not reachable or because the remote
replication slot cannot be dropped or does not exist or never existed, replication slot cannot be dropped or does not exist or never existed,
the <command>DROP SUBSCRIPTION</command> command will fail. To proceed in the <command>DROP SUBSCRIPTION</command> command will fail. To proceed
this situation, disassociate the subscription from the replication slot by in this situation, first disable the subscription by executing
executing <literal>ALTER SUBSCRIPTION ... SET (slot_name = NONE)</literal>. <literal>ALTER SUBSCRIPTION ... DISABLE</literal>, and then disassociate
it from the replication slot by executing
<literal>ALTER SUBSCRIPTION ... SET (slot_name = NONE)</literal>.
After that, <command>DROP SUBSCRIPTION</command> will no longer attempt any After that, <command>DROP SUBSCRIPTION</command> will no longer attempt any
actions on a remote host. Note that if the remote replication slot still actions on a remote host. Note that if the remote replication slot still
exists, it (and any related table synchronization slots) should then be exists, it (and any related table synchronization slots) should then be

3
src/backend/commands/subscriptioncmds.c
View File

@ -2185,7 +2185,8 @@ ReportSlotConnectionError(List *rstates, Oid subid, char *slotname, char *err)
errmsg("could not connect to publisher when attempting to drop replication slot \"%s\": %s", errmsg("could not connect to publisher when attempting to drop replication slot \"%s\": %s",
slotname, err), slotname, err),
/* translator: %s is an SQL ALTER command */ /* translator: %s is an SQL ALTER command */
errhint("Use %s to disassociate the subscription from the slot.", errhint("Use %s to disable the subscription, and then use %s to disassociate it from the slot.",
"ALTER SUBSCRIPTION ... DISABLE",
"ALTER SUBSCRIPTION ... SET (slot_name = NONE)"))); "ALTER SUBSCRIPTION ... SET (slot_name = NONE)")));
} }