Re: [HACKERS] Getting error at the time of dropping subscription andfew more issues - Mailing list pgsql-hackers

From Masahiko Sawada
Subject Re: [HACKERS] Getting error at the time of dropping subscription andfew more issues
Date
Msg-id CAD21AoCHgZHia-8bCgfsC0PPUsQ6Pw+XrAh0+qT9o7Zc48ANMQ@mail.gmail.com
Whole thread Raw
In response to Re: [HACKERS] Getting error at the time of dropping subscription andfew more issues  (Peter Eisentraut <peter.eisentraut@2ndquadrant.com>)
List pgsql-hackers
On Fri, May 19, 2017 at 9:54 AM, Peter Eisentraut
<peter.eisentraut@2ndquadrant.com> wrote:
> On 5/12/17 13:25, Masahiko Sawada wrote:
>>> postgres=#  alter subscription sub set publication pub refresh;
>>> NOTICE:  removed subscription for table public.t1
>>> NOTICE:  removed subscription for table public.t2
>>> ALTER SUBSCRIPTION
>>>
>>> I think  - in publication too ,we should provide NOTICE messages.
>>>
>> I guess one of the reason why we emit such a NOTICE message is that
>> subscriber cannot control which table the upstream server replicate.
>> So if a table got disassociated on the publisher the subscriber should
>> report that to user. On the other hand, since the publication can
>> control it and the changes are obvious, I'm not sure we really need to
>> do that.
>>
>> BTW I think it's better for the above NOTICE message to have subscription name.
>
> Why?  These come directly has a result of the ALTER SUBSCRIPTION
> command, so you see what they refer to.
>

Because I thought it would be helpful for DBA when the log is appeared
in server log. It doesn't appear by default though.

Regards,

--
Masahiko Sawada
NIPPON TELEGRAPH AND TELEPHONE CORPORATION
NTT Open Source Software Center



pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: [HACKERS] Re: proposal - using names as primary names of plpgsqlfunction parameters instead $ based names
Next
From: Peter Eisentraut
Date:
Subject: Re: [HACKERS] Fix refresh_option syntax of ALTER SUBSCRIPTION indocument