From: | Peter Smith <smithpb2250(at)gmail(dot)com> |
---|---|
To: | vignesh C <vignesh21(at)gmail(dot)com> |
Cc: | "Hayato Kuroda (Fujitsu)" <kuroda(dot)hayato(at)fujitsu(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org>, Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com> |
Subject: | Re: Documentation to upgrade logical replication cluster |
Date: | 2024-02-22 04:05:16 |
Message-ID: | CAHut+PuDLX1wbgAkmH54XmXr7vZxGxEOL7OdJj7P5dS2jZFZdw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Here are some minor comments for patch v8-0001.
======
doc/src/sgml/glossary.sgml
1.
+ <glossdef>
+ <para>
+ A set of publisher and subscriber instances with publisher instance
+ replicating changes to the subscriber instance.
+ </para>
+ </glossdef>
/with publisher instance/with the publisher instance/
~~~
2.
There are 2 SQL fragments but they are wrapped differently (see
below). e.g. it is not clear why is the 2nd fragment wrapped since it
is shorter than the 1st. OTOH, maybe you want the 1st fragment to
wrap. Either way, consistency wrapping would be better.
postgres=# SELECT slot_name FROM pg_replication_slots WHERE slot_type
= 'logical' AND conflict_reason IS NOT NULL;
slot_name
-----------
(0 rows)
versus
SELECT count(*) FROM pg_replication_slots WHERE slot_type = 'logical'
AND temporary IS false;
======
Kind Regards,
Peter Smith.
Fujitsu Australia
From | Date | Subject | |
---|---|---|---|
Next Message | Ashutosh Bapat | 2024-02-22 04:06:06 | Re: RFC: Logging plan of the running query |
Previous Message | zwj | 2024-02-22 03:45:15 | 回复: bug report: some issues about pg_15_stable(8fa4a1ac61189efffb8b851ee77e1bc87360c445) |