Re: Enhance create subscription reference manual - Mailing list pgsql-hackers

From Amit Kapila
Subject Re: Enhance create subscription reference manual
Date
Msg-id CAA4eK1LUyN0bbBaAe7NMrHxXFKt6ncpMVXG1E2Fesgq09_DpXw@mail.gmail.com
Whole thread Raw
In response to Re: Enhance create subscription reference manual  (Tatsuo Ishii <ishii@postgresql.org>)
Responses Re: Enhance create subscription reference manual
List pgsql-hackers
On Thu, Oct 3, 2024 at 8:53 AM Tatsuo Ishii <ishii@postgresql.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?

--
With Regards,
Amit Kapila.



pgsql-hackers by date:

Previous
From: Amit Kapila
Date:
Subject: Re: Remove obsolete comment in reorderbuffer.h
Next
From: "Regina Obe"
Date:
Subject: mingw64 built pg_sphere not loadable in VC++ PostgreSQL builds - double_to_shortest_decimal_buf