Small doc patch for pg_replication_slots

Started by Thomas Reissover 11 years ago3 messages
#1Thomas Reiss
thomas.reiss@dalibo.com
1 attachment(s)

Hello,

You can find attached a small patch to fix the pg_replication_slots
documentation. The slot_type and plugin are not in the appropriate
order, slot_name and plugin have a wrong type and xmin appears two times.

Regards
--
Thomas Reiss

Attachments:

pg_replication_slots.patchtext/x-patch; name=pg_replication_slots.patchDownload
diff --git a/doc/src/sgml/catalogs.sgml b/doc/src/sgml/catalogs.sgml
index 415a3bc..d8c41d9 100644
--- a/doc/src/sgml/catalogs.sgml
+++ b/doc/src/sgml/catalogs.sgml
@@ -5252,23 +5252,23 @@
     <tbody>
      <row>
       <entry><structfield>slot_name</structfield></entry>
-      <entry><type>text</type></entry>
+      <entry><type>name</type></entry>
       <entry></entry>
       <entry>A unique, cluster-wide identifier for the replication slot</entry>
      </row>
 
      <row>
-      <entry><structfield>slot_type</structfield></entry>
-      <entry><type>text</type></entry>
+      <entry><structfield>plugin</structfield></entry>
+      <entry><type>name</type></entry>
       <entry></entry>
-      <entry>The slot type - <literal>physical</> or <literal>logical</></entry>
+      <entry>The basename of the shared object containing the output plugin this logical slot is using, or null for physical slots.</entry>
      </row>
 
      <row>
-      <entry><structfield>plugin</structfield></entry>
+      <entry><structfield>slot_type</structfield></entry>
       <entry><type>text</type></entry>
       <entry></entry>
-      <entry>The basename of the shared object containing the output plugin this logical slot is using, or null for physical slots.</entry>
+      <entry>The slot type - <literal>physical</> or <literal>logical</></entry>
      </row>
 
      <row>
@@ -5305,7 +5305,7 @@
      </row>
 
      <row>
-      <entry><structfield>xmin</structfield></entry>
+      <entry><structfield>catalog_xmin</structfield></entry>
       <entry><type>xid</type></entry>
       <entry></entry>
       <entry>The oldest transaction that this slot needs the database to
@@ -5315,14 +5315,6 @@
      </row>
 
      <row>
-      <entry><structfield>catalog_xmin</structfield></entry>
-      <entry><type>xid</type></entry>
-      <entry></entry>
-      <entry>The <literal>xmin</literal>, or oldest transaction ID, that this
-      slot forces to be retained in the system catalogs. </entry>
-     </row>
-
-     <row>
       <entry><structfield>restart_lsn</structfield></entry>
       <entry><type>pg_lsn</type></entry>
       <entry></entry>
#2Robert Haas
robertmhaas@gmail.com
In reply to: Thomas Reiss (#1)
Re: Small doc patch for pg_replication_slots

On Tue, Apr 29, 2014 at 5:39 AM, Thomas Reiss <thomas.reiss@dalibo.com> wrote:

You can find attached a small patch to fix the pg_replication_slots
documentation. The slot_type and plugin are not in the appropriate
order, slot_name and plugin have a wrong type and xmin appears two times.

Without the patch, the description of catalog_xmin is:

<entry>The <literal>xmin</literal>, or oldest transaction ID, that this
slot forces to be retained in the system catalogs. </entry>

With the patch it's:

<entry>The oldest transaction that this slot needs the database to
retain. <literal>VACUUM</literal> cannot remove catalog tuples deleted
by any later transaction.

That's only one word different from the language for xmin, which
doesn't seem quite right. Committed after fixing that.

...Robert

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

#3Thomas Reiss
thomas.reiss@dalibo.com
In reply to: Robert Haas (#2)
Re: Small doc patch for pg_replication_slots

Le 01/05/2014 04:56, Robert Haas a écrit :

On Tue, Apr 29, 2014 at 5:39 AM, Thomas Reiss <thomas.reiss@dalibo.com> wrote:

You can find attached a small patch to fix the pg_replication_slots
documentation. The slot_type and plugin are not in the appropriate
order, slot_name and plugin have a wrong type and xmin appears two times.

Without the patch, the description of catalog_xmin is:

<entry>The <literal>xmin</literal>, or oldest transaction ID, that this
slot forces to be retained in the system catalogs. </entry>

With the patch it's:

<entry>The oldest transaction that this slot needs the database to
retain. <literal>VACUUM</literal> cannot remove catalog tuples deleted
by any later transaction.

That's only one word different from the language for xmin, which
doesn't seem quite right. Committed after fixing that.

...Robert

Thanks

--
Thomas Reiss
Consultant Dalibo
http://dalibo.com - http://dalibo.org

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers