Re: Modify the document of Logical Replication configuration settings - Mailing list pgsql-hackers

From Bharath Rupireddy
Subject Re: Modify the document of Logical Replication configuration settings
Date
Msg-id CALj2ACUXrM5hsxJRBM5O+ugHcrmi3j61nQfKhzfDsmeEYUhVGA@mail.gmail.com
Whole thread Raw
In response to Modify the document of Logical Replication configuration settings  ("Takamichi Osumi (Fujitsu)" <osumi.takamichi@fujitsu.com>)
Responses Re: Modify the document of Logical Replication configuration settings
List pgsql-hackers
On Wed, Jan 18, 2023 at 12:31 PM Takamichi Osumi (Fujitsu)
<osumi.takamichi@fujitsu.com> wrote:
>
> Hi, hackers
>
> The attached patch includes below two changes for the description of
> Logical Replication "Configuration Settings".
>
> 1. Add one brief description about wal_sender_timeout.
>    I made it similar to one other sentence for subscriber.

+   <para>
+    Logical replication walsender is also affected by
+    <link linkend="guc-wal-sender-timeout"><varname>wal_sender_timeout</varname></link>.
+   </para>

Looks fine. Adding something like [1] in wal_sender_timeout GUC's
description might be a good idea just to give specific information
that the logical replication subscribers too get affected. Perhaps,
it's not required since the postgres glossary wraps logical
replication subscriber under standby anyway -
https://www.postgresql.org/docs/devel/glossary.html. To me personally,
the typical notion of standby is the one connected to primary via
streaming replication.

> 2. Fix a wrong GUC name "wal_receiver_retry_interval".
>    I think this doesn't seem to exist and would mean "wal_retrieve_retry_interval".

Good catch. +1.

[1]
diff --git a/doc/src/sgml/config.sgml b/doc/src/sgml/config.sgml
index 89d53f2a64..6f9509267c 100644
--- a/doc/src/sgml/config.sgml
+++ b/doc/src/sgml/config.sgml
@@ -4326,7 +4326,8 @@ restore_command = 'copy
"C:\\server\\archivedir\\%f" "%p"'  # Windows
        <para>
         Terminate replication connections that are inactive for longer
         than this amount of time. This is useful for
-        the sending server to detect a standby crash or network outage.
+        the sending server to detect a standby crash or logical replication
+        subscriber crash or network outage.
         If this value is specified without units, it is taken as milliseconds.
         The default value is 60 seconds.
         A value of zero disables the timeout mechanism.

--
Bharath Rupireddy
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com



pgsql-hackers by date:

Previous
From: Nikita Malakhov
Date:
Subject: Re: Inconsistency in vacuum behavior
Next
From: Michael Paquier
Date:
Subject: Issue with psql's create_help.pl under perlcritic