Here are some comments for patch v3:
1.
+ <glossentry id="glossary-publication-node">
+ <glossterm>Publication node</glossterm>
+ <glossdef>
+ <para>
+ A <glossterm linkend="glossary-instance">node</glossterm> where a
+ <glossterm
linkend="glossary-publication">publication</glossterm> is defined
+ for <glossterm linkend="glossary-replication">logical
replication</glossterm>.
+ </para>
+ </glossdef>
+ </glossentry>
+
I felt the word "node" here should link to the glossary term "Node",
instead of directly to the term "Instance".
~~
2.
+ <glossentry id="glossary-subscription-node">
+ <glossterm>Subscription node</glossterm>
+ <glossdef>
+ <para>
+ A <glossterm linkend="glossary-instance">node</glossterm> where a
+ <glossterm
linkend="glossary-subscription">subscription</glossterm> is defined
+ for <glossterm linkend="glossary-replication">logical
replication</glossterm>.
+ </para>
+ </glossdef>
+ </glossentry>
+
(same comment as above)
I felt the word "node" here should link to the glossary term "Node",
instead of directly to the term "Instance".
~~
Apart from those links, it looks good to me. Let's see what others think.
======
Kind Regards,
Peter Smith.
Fujitsu Australia