pgsql: Be precise about when ALTER SUBSCRIPTION needs conninfo.

From: Jeff Davis <jdavis(at)postgresql(dot)org>
To: pgsql-committers(at)lists(dot)postgresql(dot)org
Subject: pgsql: Be precise about when ALTER SUBSCRIPTION needs conninfo.
Date: 2026-08-05 20:13:38
Message-ID: E1wrhze-00000000LAQ-0tjw@gemulon.postgresql.org
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

Be precise about when ALTER SUBSCRIPTION needs conninfo.

Decide early whether the original conninfo is needed so that errors
happen consistently.

Addresses finding 12 in report from linked discussion.

Co-authored-by: Shlok Kyal <shlok(dot)kyal(dot)oss(at)gmail(dot)com>
Reported-by: Noah Misch <noah(at)leadboat(dot)com>
Reviewed-by: Hayato Kuroda (Fujitsu) <kuroda(dot)hayato(at)fujitsu(dot)com>
Reviewed-by: Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>
Discussion: https://postgr.es/m/20260710195902.4f.noahmisch%40microsoft.com
Backpatch-through: 19

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/e90e198e3f478ebc0b1fba0347ad9d76c48e7fa3

Modified Files
--------------
src/backend/commands/subscriptioncmds.c | 79 +++++++++++++++++++-----------
src/test/regress/expected/subscription.out | 8 +++
src/test/regress/sql/subscription.sql | 9 ++++
3 files changed, 68 insertions(+), 28 deletions(-)

Browse pgsql-committers by date

  From Date Subject
Next Message Jeff Davis 2026-08-05 20:14:10 pgsql: Be precise about when ALTER SUBSCRIPTION needs conninfo.
Previous Message Jeff Davis 2026-08-05 19:58:55 pgsql: Build subscription conninfo after checking that it's enabled.