Re: Improve errhint for ALTER SUBSCRIPTION ADD/DROP PUBLICATION
Amit Kapila <amit.kapila16@gmail.com>
From: Amit Kapila <amit.kapila16@gmail.com>
To: "houzj.fnst@fujitsu.com" <houzj.fnst@fujitsu.com>
Cc: PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2022-10-17T04:13:14Z
Lists: pgsql-hackers
On Mon, Oct 17, 2022 at 8:39 AM houzj.fnst@fujitsu.com <houzj.fnst@fujitsu.com> wrote: > > While working on some logical replication related features. > I found the HINT message could be improved when I tried to add a publication to > a subscription which was disabled. > > alter subscription sub add publication pub2; > -- > ERROR: ALTER SUBSCRIPTION with refresh is not allowed for disabled subscriptions > HINT: Use ALTER SUBSCRIPTION ... SET PUBLICATION ... WITH (refresh = false). > -- > > Because I was executing the ADD PUBLICATION command, I feel the hint should > also mention it instead of SET PUBLICATION. > +1. I haven't tested it yet but the changes look sane. -- With Regards, Amit Kapila.
Commits
-
Improve errhint for ALTER SUBSCRIPTION ADD/DROP PUBLICATION
- 915a6c4e22ec 16.0 landed
- 25fb9579bbb9 15.1 landed