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
