Here are some review comments for patch v43-0002:
======
1. doc/src/sgml/ref/create_subscription.sgml
@@ -403,7 +403,9 @@ CREATE SUBSCRIPTION <replaceable
class="parameter">subscription_name</replaceabl
warning to notify the user to check the publisher tables. Before continuing
with other operations the user should check that publisher tables did not
have data with different origins to prevent data inconsistency issues on the
- subscriber.
+ subscriber. Refer to <xref linkend="replication-between-primaries"/> for
+ how <literal>copy_data</literal> and <literal>origin</literal> can be used
+ to set up replication between primaries.
</para>
Regarding my earlier v43-0001 review (see [1] comment #2) perhaps
another pg docs section should be added in the
logical-replication.sgml (e.g. "Specifying origins during CREATE
SUBSCRIPTION"), so then this Notes text also should have more added to
it.
SUGGESTION
Refer to <XXX_REF> for details about potential initialization
inconsistency warnings using origin=NONE.
Refer to <YYY_REF> for how copy_data and origin can be used to set up
replication between primaries.
------
[1] https://www.postgresql.org/message-id/CAHut%2BPvonTd423-cWqoxh0w8Bd_Po3OToqqyxuR1iMNmxSLr_Q%40mail.gmail.com
Kind Regards,
Peter Smith.
Fujitsu Australia
