From: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
---|---|
To: | Tatsuo Ishii <ishii(at)postgresql(dot)org> |
Cc: | nagata(at)sraoss(dot)co(dot)jp, david(dot)g(dot)johnston(at)gmail(dot)com, smithpb2250(at)gmail(dot)com, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Enhance create subscription reference manual |
Date: | 2024-10-18 10:04:12 |
Message-ID: | CAA4eK1Lq5poHvrTb7Goba86Rq-ay4WSOvEyEf3U49MoWV7Zkiw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Fri, Oct 18, 2024 at 9:46 AM Tatsuo Ishii <ishii(at)postgresql(dot)org> wrote:
>
> Hi Amit,
>
> > On Thu, Oct 3, 2024 at 8:53 AM Tatsuo Ishii <ishii(at)postgresql(dot)org> wrote:
> >>
> >> > parameter in this case (it is an "optional" parameter, though). However,
> >> > when we refer to the stored catalog value, we should call it an option or
> >> > a property and calling it parameter is not suitable.
> >>
> >> Not sure. The stored catalog value of a subscription can be changed
> >> ALTER SUBSCRIPTION. In the ALTER SUBSCRIPTION manual, the placeholders
> >> for these properties are "parameter". So I think we should use
> >> "parameter" in this case at least for the stored catalog values of
> >> subscriptions.
> >>
> >> > If so, I feel that "the failover" in the following statement means
> >> > the catalog value (or the failover feature itself), so we should not
> >> > rewrite this to "the failover parameter".
> >>
> >> My conclusion is we should rewrite it as "the failover parameter" for
> >> the reason above.
> >>
> >> >> To initiate replication, you must manually create the replication slot,
> >> >> enable the failover if required, enable the subscription, and refresh the
> >> >> subscription.
> >> >
> >> > Instead, should we use "failover option"?
> >>
> >
> > Sounds reasonable to me. Would you like to propose the patch with the
> > changes you have in mind?
>
> Thanks for the comment. I would like to propose the patch. But I am
> not sure which you feel resonable with "the failover parameter" or
> "the failover option"?
>
"the failover option".
--
With Regards,
Amit Kapila.
From | Date | Subject | |
---|---|---|---|
Next Message | Seino Yuki | 2024-10-18 10:07:28 | Re: Add “FOR UPDATE NOWAIT” lock details to the log. |
Previous Message | Daniel Gustafsson | 2024-10-18 09:59:48 | Re: Use more CppAsString2() in pg_amcheck.c |