From: | Michael Paquier <michael(at)paquier(dot)xyz> |
---|---|
To: | Marina Polyakova <m(dot)polyakova(at)postgrespro(dot)ru> |
Cc: | daniel(at)yesql(dot)se, a(dot)lakhin(at)postgrespro(dot)ru, pgsql-docs(at)postgresql(dot)org |
Subject: | Re: Missing comma? |
Date: | 2020-05-14 00:07:38 |
Message-ID: | 20200514000738.GA166343@paquier.xyz |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-docs |
On Wed, May 13, 2020 at 05:17:43PM +0300, Marina Polyakova wrote:
> in my opinion it's a little unclear what "or default" means at the end,
> because the comma is used to separate enumeration elements ("d = default
> <...>, n = nothing, f = all columns, i = index <...>") and inside the
> element description ("i = index with indisreplident set, or default").
> Therefore here is an additional patch from me to clarify this place, thanks
> to Alexander Lakhin for help.
Yes, I agree that this last "or default" in the docs does not make
much sense, because we always enforce REPLICA_IDENTITY_NOTHING if
there is no replica identity.
> diff --git a/doc/src/sgml/catalogs.sgml b/doc/src/sgml/catalogs.sgml
> index 02ddebae99d98110a8dd290dd4cb0c980adf7984..034a08f80ea4269f131e7e1383ba482fd76d9344 100644
> --- a/doc/src/sgml/catalogs.sgml
> +++ b/doc/src/sgml/catalogs.sgml
> @@ -1936,7 +1936,7 @@ SCRAM-SHA-256$<replaceable><iteration count></replaceable>:<replaceable>&l
> <literal>d</literal> = default (primary key, if any),
> <literal>n</literal> = nothing,
> <literal>f</literal> = all columns,
> - <literal>i</literal> = index with <structfield>indisreplident</structfield> set, or default
> + <literal>i</literal> = index with <structfield>indisreplident</structfield> set (if any)
> </entry>
> </row>
And you don't need the ("if any") either here, no?
REPLICA_IDENTITY_INDEX means that we have an index associated with the
replica identity so it seems to me that this last part can just be
removed.
--
Michael
From | Date | Subject | |
---|---|---|---|
Next Message | Michael Paquier | 2020-05-14 02:17:13 | Re: TLS docs fixes for <programlisting> |
Previous Message | Daniel Gustafsson | 2020-05-13 21:07:44 | TLS docs fixes for <programlisting> |