Re: Add publisher and subscriber to glossary documentation. - Mailing list pgsql-hackers

From Shlok Kyal
Subject Re: Add publisher and subscriber to glossary documentation.
Date
Msg-id CANhcyEXtym3aySz1_K3zD9oW4kX+fcoayR0QzC6W9VU7GKYV2w@mail.gmail.com
Whole thread Raw
In response to Re: Add publisher and subscriber to glossary documentation.  (Peter Smith <smithpb2250@gmail.com>)
Responses Re: Add publisher and subscriber to glossary documentation.
List pgsql-hackers
> 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

pgsql-hackers by date:

Previous
From: Ashutosh Bapat
Date:
Subject: Re: RFC: Logging plan of the running query
Next
From: Masahiko Sawada
Date:
Subject: Re: Improve eviction algorithm in ReorderBuffer