Hi, here are some review comments for the docs patch v5-0001.
Apart from these it LGTM.
======
doc/src/sgml/logical-replication.sgml
1.
+ <para>
+ On the subscriber node, use the following SQL to identify which slots
+ should be synced to the standby that we plan to promote. This query will
+ return the relevant replication slots, including the main slots and table
+ synchronization slots associated with the failover enabled subscriptions.
/failover enabled/failover-enabled/
~~~
2.
+ <para>
+ If all the slots are present on the standby server and result
+ (<literal>failover_ready</literal>) of is true, then existing subscriptions
+ can continue subscribing to publications now on the new primary server
+ without any loss of data.
+ </para>
Hmm. It looks like there is some typo or missing words here: "of is true".
Did you mean something like: "of the above SQL query is true"?
======
Kind Regards,
Peter Smith.
Fujitsu Australia
