pg.ddx.io  pgsql-docs@postgresql.org mailing list archive  
help / color / mirror / Atom feed
Minor doc fixes
7+ messages / 4 participants
[nested] [flat]

* Minor doc fixes
@ 2021-07-13 13:13 Ekaterina Kiryanova <e.kiryanova@postgrespro.ru>
  2021-07-13 21:32 ` Re: Minor doc fixes Daniel Gustafsson <daniel@yesql.se>
  0 siblings, 1 reply; 7+ messages in thread

From: Ekaterina Kiryanova @ 2021-07-13 13:13 UTC (permalink / raw)
  To: pgsql-docs@lists.postgresql.org

Hi!
I've noticed some minor doc inconsistencies.
Please review the attached patch.

-- 
Ekaterina Kiryanova
Technical Writer
Postgres Professional

Attachments:

minor-fixes.patch (text/x-patch, 2.9K) download diff --git a/doc/src/sgml/amcheck.sgml b/doc/src/sgml/amcheck.sgml index a2571d33ae6..c570690b59c 100644 --- a/doc/src/sgml/amcheck.sgml +++ b/doc/src/sgml/amcheck.sgml @@ -279,7 +279,7 @@ SET client_min_messages = DEBUG1; <para> If specified, corruption checking begins at the specified block, skipping all previous blocks. It is an error to specify a - <literal>startblock</literal> outside the range of blocks in the + <parameter>startblock</parameter> outside the range of blocks in the target table. </para> <para> @@ -293,7 +293,7 @@ SET client_min_messages = DEBUG1; <para> If specified, corruption checking ends at the specified block, skipping all remaining blocks. It is an error to specify an - <literal>endblock</literal> outside the range of blocks in the target + <parameter>endblock</parameter> outside the range of blocks in the target table. </para> <para> diff --git a/doc/src/sgml/brin.sgml b/doc/src/sgml/brin.sgml index 39c01dc6808..940563c83f9 100644 --- a/doc/src/sgml/brin.sgml +++ b/doc/src/sgml/brin.sgml @@ -716,7 +716,7 @@ LOG: request for BRIN range summarization for index "brin_wi_idx" page 128 was </para> <para> - <acronym>bloom</acronym> operator classes accept these parameters: + <literal>bloom</literal> operator classes accept these parameters: </para> <variablelist> diff --git a/doc/src/sgml/catalogs.sgml b/doc/src/sgml/catalogs.sgml index f517a7d4aff..71ff9b8cc9f 100644 --- a/doc/src/sgml/catalogs.sgml +++ b/doc/src/sgml/catalogs.sgml @@ -7403,8 +7403,8 @@ SCRAM-SHA-256$<replaceable>&lt;iteration count&gt;</replaceable>:<replaceable>&l An array containing codes for the enabled statistics kinds; valid values are: <literal>d</literal> for n-distinct statistics, - <literal>f</literal> for functional dependency statistics, and - <literal>m</literal> for most common values (MCV) list statistics + <literal>f</literal> for functional dependency statistics, + <literal>m</literal> for most common values (MCV) list statistics, and <literal>e</literal> for expression statistics </para></entry> </row> diff --git a/doc/src/sgml/config.sgml b/doc/src/sgml/config.sgml index 381d8636ab4..d64494186a7 100644 --- a/doc/src/sgml/config.sgml +++ b/doc/src/sgml/config.sgml @@ -7017,7 +7017,7 @@ local0.* /var/log/postgresql </row> <row> <entry><literal>%Q</literal></entry> - <entry>query identifier of the current query. Query + <entry>Query identifier of the current query. Query identifiers are not computed by default, so this field will be zero unless <xref linkend="guc-compute-query-id"/> parameter is enabled or a third-party module that computes
^ permalink raw reply [nested|flat] 7+ messages in thread

* Re: Minor doc fixes
  2021-07-13 13:13 Minor doc fixes Ekaterina Kiryanova <e.kiryanova@postgrespro.ru>
@ 2021-07-13 21:32 ` Daniel Gustafsson <daniel@yesql.se>
  2021-07-14 05:28   ` Re: Minor doc fixes Laurenz Albe <laurenz.albe@cybertec.at>
  2021-07-15 21:31   ` Re: Minor doc fixes Daniel Gustafsson <daniel@yesql.se>
  0 siblings, 2 replies; 7+ messages in thread

From: Daniel Gustafsson @ 2021-07-13 21:32 UTC (permalink / raw)
  To: Ekaterina Kiryanova <e.kiryanova@postgrespro.ru>; +Cc: pgsql-docs@lists.postgresql.org

> On 13 Jul 2021, at 15:13, Ekaterina Kiryanova <e.kiryanova@postgrespro.ru> wrote:

> I've noticed some minor doc inconsistencies.
> Please review the attached patch.

Thanks for your patch, reviewing it I agree with your proposed changes.  Unless
there are objections I'll apply this within a few days.

--
Daniel Gustafsson		https://vmware.com/






^ permalink  raw  reply  [nested|flat] 7+ messages in thread

* Re: Minor doc fixes
  2021-07-13 13:13 Minor doc fixes Ekaterina Kiryanova <e.kiryanova@postgrespro.ru>
  2021-07-13 21:32 ` Re: Minor doc fixes Daniel Gustafsson <daniel@yesql.se>
@ 2021-07-14 05:28   ` Laurenz Albe <laurenz.albe@cybertec.at>
  1 sibling, 0 replies; 7+ messages in thread

From: Laurenz Albe @ 2021-07-14 05:28 UTC (permalink / raw)
  To: Daniel Gustafsson <daniel@yesql.se>; Ekaterina Kiryanova <e.kiryanova@postgrespro.ru>; +Cc: pgsql-docs@lists.postgresql.org

On Tue, 2021-07-13 at 23:32 +0200, Daniel Gustafsson wrote:
> > On 13 Jul 2021, at 15:13, Ekaterina Kiryanova <e.kiryanova@postgrespro.ru> wrote:
> > I've noticed some minor doc inconsistencies.
> > Please review the attached patch.
> 
> Thanks for your patch, reviewing it I agree with your proposed changes.  Unless
> there are objections I'll apply this within a few days.

The changes look good to me as well.

Yours,
Laurenz Albe






^ permalink  raw  reply  [nested|flat] 7+ messages in thread

* Re: Minor doc fixes
  2021-07-13 13:13 Minor doc fixes Ekaterina Kiryanova <e.kiryanova@postgrespro.ru>
  2021-07-13 21:32 ` Re: Minor doc fixes Daniel Gustafsson <daniel@yesql.se>
@ 2021-07-15 21:31   ` Daniel Gustafsson <daniel@yesql.se>
  1 sibling, 0 replies; 7+ messages in thread

From: Daniel Gustafsson @ 2021-07-15 21:31 UTC (permalink / raw)
  To: Ekaterina Kiryanova <e.kiryanova@postgrespro.ru>; +Cc: pgsql-docs@lists.postgresql.org

> On 13 Jul 2021, at 23:32, Daniel Gustafsson <daniel@yesql.se> wrote:
> 
>> On 13 Jul 2021, at 15:13, Ekaterina Kiryanova <e.kiryanova@postgrespro.ru> wrote:
> 
>> I've noticed some minor doc inconsistencies.
>> Please review the attached patch.
> 
> Thanks for your patch, reviewing it I agree with your proposed changes.  Unless
> there are objections I'll apply this within a few days.

Applied to with a small edit to make the bloom operator class mention the same
as the minmax-multi, which previously also had an acronym tag (fixed in
fb72a7b8c3).  I’ve backpatched it to v14 to match fb72a7b8c3.  Thanks!

--
Daniel Gustafsson		https://vmware.com/






^ permalink  raw  reply  [nested|flat] 7+ messages in thread

* Minor doc fixes
@ 2021-09-27 08:42 Ekaterina Kiryanova <e.kiryanova@postgrespro.ru>
  2021-09-27 09:07 ` Re: Minor doc fixes Michael Paquier <michael@paquier.xyz>
  0 siblings, 1 reply; 7+ messages in thread

From: Ekaterina Kiryanova @ 2021-09-27 08:42 UTC (permalink / raw)
  To: pgsql-docs@lists.postgresql.org

Hi!
I've come across some minor inconsistencies in PostgreSQL 14 docs.
Please review the attached patch.

-- 
Ekaterina Kiryanova
Technical Writer
Postgres Professional
file:///home/ladyune/patches/minor_doc_fixes.patch

Attachments:

minor_doc_fixes.patch (text/x-patch, 6.9K) download diff --git a/doc/src/sgml/btree.sgml b/doc/src/sgml/btree.sgml index f569f93f352..a9200ee52e6 100644 --- a/doc/src/sgml/btree.sgml +++ b/doc/src/sgml/btree.sgml @@ -664,7 +664,7 @@ options(<replaceable>relopts</replaceable> <type>local_relopts *</type>) returns certain implementation-level heuristics will fail to identify and delete even one garbage index tuple (in which case a page split or deduplication pass resolves the issue of an incoming new tuple not - fitting on a leaf page). The worst case number of versions that + fitting on a leaf page). The worst-case number of versions that any index scan must traverse (for any single logical row) is an important contributor to overall system responsiveness and throughput. A bottom-up index deletion pass targets suspected @@ -706,7 +706,7 @@ options(<replaceable>relopts</replaceable> <type>local_relopts *</type>) returns This is expected with any B-Tree index that is subject to significant version churn from <command>UPDATE</command>s that rarely or never logically modify the columns that the index covers. - The average and worst case number of versions per logical row can + The average and worst-case number of versions per logical row can be kept low purely through targeted incremental deletion passes. It's quite possible that the on-disk size of certain indexes will never increase by even one single page/block despite @@ -811,7 +811,7 @@ options(<replaceable>relopts</replaceable> <type>local_relopts *</type>) returns constraints) to use deduplication. This allows leaf pages to temporarily <quote>absorb</quote> extra version churn duplicates. Deduplication in unique indexes augments bottom-up index deletion, - especially in cases where a long-running transactions holds a + especially in cases where a long-running transaction holds a snapshot that blocks garbage collection. The goal is to buy time for the bottom-up index deletion strategy to become effective again. Delaying page splits until a single long-running diff --git a/doc/src/sgml/catalogs.sgml b/doc/src/sgml/catalogs.sgml index 2f0def9b19a..0e590059e89 100644 --- a/doc/src/sgml/catalogs.sgml +++ b/doc/src/sgml/catalogs.sgml @@ -13116,7 +13116,7 @@ SELECT * FROM pg_locks pl LEFT JOIN pg_prepared_xacts ppx <structname>pg_stats_ext_exprs</structname> is also designed to present the information in a more readable format than the underlying catalogs &mdash; at the cost that its schema must be extended whenever the structure - of statistics in <link linkend="catalog-pg-statistic"><structname>pg_statistic</structname></link> changes. + of statistics in <link linkend="catalog-pg-statistic-ext"><structname>pg_statistic_ext</structname></link> changes. </para> <table> diff --git a/doc/src/sgml/ref/alter_subscription.sgml b/doc/src/sgml/ref/alter_subscription.sgml index bec5e9c483f..db9713e6667 100644 --- a/doc/src/sgml/ref/alter_subscription.sgml +++ b/doc/src/sgml/ref/alter_subscription.sgml @@ -65,8 +65,8 @@ ALTER SUBSCRIPTION <replaceable class="parameter">name</replaceable> RENAME TO < <para> Commands <command>ALTER SUBSCRIPTION ... REFRESH PUBLICATION</command> and - <command>ALTER SUBSCRIPTION ... {SET|ADD|DROP} PUBLICATION ...</command> with refresh - option as true cannot be executed inside a transaction block. + <command>ALTER SUBSCRIPTION ... {SET|ADD|DROP} PUBLICATION ...</command> with <literal>refresh</literal> + option as <literal>true</literal> cannot be executed inside a transaction block. These commands also cannot be executed when the subscription has <literal>two_phase</literal> commit enabled, diff --git a/doc/src/sgml/ref/commit.sgml b/doc/src/sgml/ref/commit.sgml index 5f244cdd3c1..43f9e984bcc 100644 --- a/doc/src/sgml/ref/commit.sgml +++ b/doc/src/sgml/ref/commit.sgml @@ -97,7 +97,7 @@ COMMIT; <para> The command <command>COMMIT</command> conforms to the SQL standard. The - form <literal>COMMIT TRANSACTION</literal> is a PostgreSQL extension. + form <literal>COMMIT TRANSACTION</literal> is a <productname>PostgreSQL</productname> extension. </para> </refsect1> diff --git a/doc/src/sgml/ref/initdb.sgml b/doc/src/sgml/ref/initdb.sgml index 3d888387126..9d47cacbe0b 100644 --- a/doc/src/sgml/ref/initdb.sgml +++ b/doc/src/sgml/ref/initdb.sgml @@ -283,7 +283,7 @@ PostgreSQL documentation By default, <command>initdb</command> will write instructions for how to start the cluster at the end of its output. This option causes those instructions to be left out. This is primarily intended for use - by tools that wrap <command>initdb</command> in platform specific + by tools that wrap <command>initdb</command> in platform-specific behavior, where those instructions are likely to be incorrect. </para> </listitem> diff --git a/doc/src/sgml/ref/psql-ref.sgml b/doc/src/sgml/ref/psql-ref.sgml index fcab5c0d51a..5ef401916f2 100644 --- a/doc/src/sgml/ref/psql-ref.sgml +++ b/doc/src/sgml/ref/psql-ref.sgml @@ -1936,9 +1936,9 @@ testdb=&gt; <para> The status of each kind of extended statistics is shown in a column named after its statistic kind (e.g. Ndistinct). - "defined" means that it was requested when creating the statistics, + <literal>defined</literal> means that it was requested when creating the statistics, and NULL means it wasn't requested. - You can use pg_stats_ext if you'd like to know whether <link linkend="sql-analyze"> + You can use <structname>pg_stats_ext</structname> if you'd like to know whether <link linkend="sql-analyze"> <command>ANALYZE</command></link> was run and statistics are available to the planner. </para> diff --git a/doc/src/sgml/spgist.sgml b/doc/src/sgml/spgist.sgml index 18f1f3cdbd8..b8b6d6b8e7d 100644 --- a/doc/src/sgml/spgist.sgml +++ b/doc/src/sgml/spgist.sgml @@ -875,7 +875,7 @@ typedef struct spgLeafConsistentOut <para> Note: the <function>compress</function> method is only applied to - values to be stored. The consistent methods receive query scankeys + values to be stored. The consistent methods receive query <structfield>scankeys</structfield> unchanged, without transformation using <function>compress</function>. </para> </listitem> diff --git a/doc/src/sgml/test-decoding.sgml b/doc/src/sgml/test-decoding.sgml index fe7c9783fac..cf736bd41af 100644 --- a/doc/src/sgml/test-decoding.sgml +++ b/doc/src/sgml/test-decoding.sgml @@ -40,8 +40,8 @@ postgres=# SELECT * FROM pg_logical_slot_get_changes('test_slot', NULL, NULL, 'i </para> <para> - We can also get the changes of the in-progress transaction and the typical - output, might be: + We can also get the changes of the in-progress transaction, and the typical + output might be: <programlisting> postgres[33712]=#* SELECT * FROM pg_logical_slot_get_changes('test_slot', NULL, NULL, 'stream-changes', '1');
^ permalink raw reply [nested|flat] 7+ messages in thread

* Re: Minor doc fixes
  2021-09-27 08:42 Minor doc fixes Ekaterina Kiryanova <e.kiryanova@postgrespro.ru>
@ 2021-09-27 09:07 ` Michael Paquier <michael@paquier.xyz>
  2021-09-29 02:57   ` Re: Minor doc fixes Michael Paquier <michael@paquier.xyz>
  0 siblings, 1 reply; 7+ messages in thread

From: Michael Paquier @ 2021-09-27 09:07 UTC (permalink / raw)
  To: Ekaterina Kiryanova <e.kiryanova@postgrespro.ru>; +Cc: pgsql-docs@lists.postgresql.org

On Mon, Sep 27, 2021 at 11:42:40AM +0300, Ekaterina Kiryanova wrote:
> I've come across some minor inconsistencies in PostgreSQL 14 docs.
> Please review the attached patch.

Thanks.  These are good catches.
--
Michael

Attachments:

  signature.asc (application/pgp-signature, 832B) download

^ permalink  raw  reply  [nested|flat] 7+ messages in thread

* Re: Minor doc fixes
  2021-09-27 08:42 Minor doc fixes Ekaterina Kiryanova <e.kiryanova@postgrespro.ru>
  2021-09-27 09:07 ` Re: Minor doc fixes Michael Paquier <michael@paquier.xyz>
@ 2021-09-29 02:57   ` Michael Paquier <michael@paquier.xyz>
  0 siblings, 0 replies; 7+ messages in thread

From: Michael Paquier @ 2021-09-29 02:57 UTC (permalink / raw)
  To: Ekaterina Kiryanova <e.kiryanova@postgrespro.ru>; +Cc: pgsql-docs@lists.postgresql.org

On Mon, Sep 27, 2021 at 06:07:57PM +0900, Michael Paquier wrote:
> Thanks.  These are good catches.

Now that 14.0 has been tagged, applied.
--
Michael

Attachments:

  signature.asc (application/pgp-signature, 832B) download

^ permalink  raw  reply  [nested|flat] 7+ messages in thread


end of thread, other threads:[~2021-09-29 02:57 UTC | newest]

Thread overview: 7+ messages (download: mbox mbox.gz follow: Atom feed)
-- links below jump to the message on this page --
2021-07-13 13:13 Minor doc fixes Ekaterina Kiryanova <e.kiryanova@postgrespro.ru>
2021-07-13 21:32 ` Daniel Gustafsson <daniel@yesql.se>
2021-07-14 05:28   ` Laurenz Albe <laurenz.albe@cybertec.at>
2021-07-15 21:31   ` Daniel Gustafsson <daniel@yesql.se>
2021-09-27 08:42 Minor doc fixes Ekaterina Kiryanova <e.kiryanova@postgrespro.ru>
2021-09-27 09:07 ` Michael Paquier <michael@paquier.xyz>
2021-09-29 02:57   ` Michael Paquier <michael@paquier.xyz>

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox