From: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
---|---|
To: | Peter Smith <smithpb2250(at)gmail(dot)com> |
Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Alvaro Herrera <alvherre(at)alvh(dot)no-ip(dot)org>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: create subscription - improved warning message |
Date: | 2022-10-17 11:13:52 |
Message-ID: | CAA4eK1+cUU9+exhuEH+dMmF6q2qGmrg59n5CNVOXyzJDP9KTAw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Mon, Oct 17, 2022 at 7:17 AM Peter Smith <smithpb2250(at)gmail(dot)com> wrote:
>
>
> Updated as sugggested.
>
+ <para>
+ Sometimes, either by choice (e.g. <literal>create_slot = false</literal>),
+ or by necessity (e.g. <literal>connect = false</literal>), the remote
+ replication slot is not created automatically during
+ <literal>CREATE SUBSCRIPTION</literal>. In these cases the user will have
+ to create the slot manually before the subscription can be activated.
+ </para>
This part looks a bit odd when in the previous section we have
explained the same thing in different words. I think it may be better
if we start with something like: "As mentioned in the previous
section, there are cases where we need to create the slot manually
before the subscription can be activated.". I think you can even
combine the next para in the patch with this one.
Also, it looks odd that the patch uses examples to demonstrate how to
manually create a slot, and then we have a separate section whose
title is Examples. I am not sure what is the best way to arrange docs
here but maybe we can consider renaming the Examples section to
something more specific.
--
With Regards,
Amit Kapila.
From | Date | Subject | |
---|---|---|---|
Next Message | kuroda.hayato@fujitsu.com | 2022-10-17 12:04:07 | RE: [Proposal] Add foreign-server health checks infrastructure |
Previous Message | Richard Guo | 2022-10-17 10:44:24 | Re: remove no longer necessary Perl compatibility hack |