> 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".

I have addressed the comments and have attached the updated version.

Thanks and Regards,
Shlok Kyal

Attachment: v4-0001-Add-publisher-and-subscriber-to-glossary-document.patch
Description: Binary data

Reply via email to