From: Amit Kapila Date: Wed, 24 Dec 2025 09:55:03 +0000 (+0000) Subject: Update comments to reflect changes in 8e0d32a4a1. X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=refs%2Fheads%2FREL_17_STABLE;p=thirdparty%2Fpostgresql.git Update comments to reflect changes in 8e0d32a4a1. Commit 8e0d32a4a1 fixed an issue by allowing the replication origin to be created while marking the table sync state as SUBREL_STATE_DATASYNC. Update the comment in check_old_cluster_subscription_state() to accurately describe this corrected behavior. Author: Amit Kapila Reviewed-by: Michael Paquier Backpatch-through: 17, where the code was introduced Discussion: https://postgr.es/m/CAA4eK1+KaSf5nV_tWy+SDGV6MnFnKMhdt41jJjSDWm6yCyOcTw@mail.gmail.com Discussion: https://postgr.es/m/aUTekQTg4OYnw-Co@paquier.xyz --- diff --git a/src/bin/pg_upgrade/check.c b/src/bin/pg_upgrade/check.c index b4b596872a6..084415feadc 100644 --- a/src/bin/pg_upgrade/check.c +++ b/src/bin/pg_upgrade/check.c @@ -2055,9 +2055,9 @@ check_old_cluster_subscription_state(void) * other states listed below are not supported: * * a) SUBREL_STATE_DATASYNC: A relation upgraded while in this state - * would retain a replication slot, which could not be dropped by the - * sync worker spawned after the upgrade because the subscription ID - * used for the slot name won't match anymore. + * would retain a replication slot and origin. The sync worker spawned + * after the upgrade cannot be drop then because the subscription ID + * used for the slot and origin name no longer matches. * * b) SUBREL_STATE_SYNCDONE: A relation upgraded while in this state * would retain the replication origin when there is a failure in