Re: [DOC] Update ALTER SUBSCRIPTION documentation v2

From: Peter Smith <smithpb2250(at)gmail(dot)com>
To: Robert Sjöblom <robert(dot)sjoblom(at)fortnox(dot)se>
Cc: pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: [DOC] Update ALTER SUBSCRIPTION documentation v2
Date: 2023-05-15 23:43:37
Message-ID: CAHut+Ps21FCx2XX_PfXzqEZY13Qbo6VbZ=ynVVmL0CG1nAdsPw@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Mon, May 15, 2023 at 11:36 PM Robert Sjöblom
<robert(dot)sjoblom(at)fortnox(dot)se> wrote:
>
>
>
> On 2023-05-05 15:17, Robert Sjöblom wrote:
> >
> > Hi,
> >
> > We have recently used the PostgreSQL documentation when setting up our
> > logical replication. We noticed there was a step missing in the
> > documentation on how to drop a logical replication subscription with a
> > replication slot attached.
>
> Following discussions, please see revised documentation patch.
>

LGTM.

BTW, in the previous thread, there was also a suggestion from Amit [1]
to change the errhint in a similar way. There was no reply to Amit's
idea, so it's not clear whether it's an accidental omission from your
v2 patch or not.

------
[1] https://www.postgresql.org/message-id/CAA4eK1J11phiaoCOmsjNqPZ9BOWyLXYrfgrm5vU2uCFPF2kN1Q%40mail.gmail.com

Kind Regards,
Peter Smith.
Fujitsu Australia

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Andres Freund 2023-05-15 23:43:56 Re: issue with meson builds on msys2
Previous Message Michael Paquier 2023-05-15 23:19:04 Re: Conflict between regression tests namespace & transactions due to recent changes