Re: Add publisher and subscriber to glossary documentation.
От | Peter Smith |
---|---|
Тема | Re: Add publisher and subscriber to glossary documentation. |
Дата | |
Msg-id | CAHut+Ps0_4zU1DVtMoESFk113ASORhmi6s-u+K1LW9bv0e0VhQ@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: Add publisher and subscriber to glossary documentation. (Shlok Kyal <shlok.kyal.oss@gmail.com>) |
Ответы |
Re: Add publisher and subscriber to glossary documentation.
|
Список | pgsql-hackers |
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
В списке pgsql-hackers по дате отправления: