From: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
---|---|
To: | "houzj(dot)fnst(at)fujitsu(dot)com" <houzj(dot)fnst(at)fujitsu(dot)com> |
Cc: | PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Improve errhint for ALTER SUBSCRIPTION ADD/DROP PUBLICATION |
Date: | 2022-10-17 04:13:14 |
Message-ID: | CAA4eK1JzWLpmAGcJ4W6MnE5GEj8BWjbiQEWJGSAT2=HW47wZ9Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Oct 17, 2022 at 8:39 AM houzj(dot)fnst(at)fujitsu(dot)com
<houzj(dot)fnst(at)fujitsu(dot)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.
From | Date | Subject | |
---|---|---|---|
Next Message | Amit Kapila | 2022-10-17 04:28:31 | Re: Improve description of XLOG_RUNNING_XACTS |
Previous Message | Tomas Vondra | 2022-10-17 03:43:40 | Re: PATCH: Using BRIN indexes for sorted output |