Hi Shubham.

The patch v6-0001 LGTM.

OTOH, if you want to be picky, the docs wording could be slightly
modified to be more consistent with the coded warning message.

CURRENT
Replication failures can occur if required WAL files are prematurely
deleted. To prevent this, the source server must set <xref
linkend="guc-max-slot-wal-keep-size"/> to <literal>-1</literal>,
ensuring WAL files are not automatically removed.

SUGGESTION
Replication failures can occur if required WAL files are missing. To
prevent this, the source server must set <xref
linkend="guc-max-slot-wal-keep-size"/> to <literal>-1</literal> to
ensure that required WAL files are not prematurely removed.

~~~

See the attached NITPICKS diff if you want to make this change.

======
Kind Regards,
Peter Smith.
Fujitsu Australia
diff --git a/doc/src/sgml/ref/pg_createsubscriber.sgml 
b/doc/src/sgml/ref/pg_createsubscriber.sgml
index c5ee5f4..40b8fac 100644
--- a/doc/src/sgml/ref/pg_createsubscriber.sgml
+++ b/doc/src/sgml/ref/pg_createsubscriber.sgml
@@ -315,10 +315,10 @@ PostgreSQL documentation
    </para>
 
    <para>
-   Replication failures can occur if required WAL files are prematurely 
deleted.
-   To prevent this, the source server must set <xref
-   linkend="guc-max-slot-wal-keep-size"/>  to <literal>-1</literal>, ensuring
-   WAL files are not automatically removed.
+   Replication failures can occur if required WAL files are missing. To prevent
+   this, the source server must set <xref 
linkend="guc-max-slot-wal-keep-size"/>
+   to <literal>-1</literal> to ensure that required WAL files are not
+   prematurely removed.
    </para>
   </refsect2>
 

Reply via email to