Please find a few more comments on July4 patch 6) + <para> + To synchronize sequences from a publisher to a subscriber, first publish + them using <link linkend="sql-createpublication-params-for-all-sequences"> + <command>CREATE PUBLICATION ... FOR ALL SEQUENCES</command></link>. + </para>
This sentence looks odd, as we have 'first' but no follow-up sentence after that. Can we please combine this line with the next one in the doc saying: To synchronize sequences from a publisher to a subscriber, first publish them using CREATE PUBLICATION ... FOR ALL SEQUENCES and then at the subscriber side: 7) + <para> + This parameter is not applicable for sequences. + </para> It is mentioned 3 times in doc for publish, publish_generated_columns and publish_via_partition_root. Instead shall we mention it once for WITH-clause itself. Something like: This clause specifies optional parameters for a publication when publishing tables. This clause is not applicable for sequences. 8) + The view <structname>pg_publication_sequences</structname> provides + information about the mapping between publications and information of + sequences they contain. Why not: "The view pg_publication_sequences provides information about the mapping between publications and sequences." I think the existing detail has been written similar to 'pg_publication_tables' doc. But there, 'information of tables' made sense as pg_publication_tables has attnames and rowfilters too. But pg_publication_sequences OTOH just has the mapping between names. No other information. 9) + <sect2 id="sequence-definition-mismatches"> + <title>Sequence Definition Mismatches</title> + <warning> + <para> + During sequence synchronization, the sequence definitions of the publisher + and the subscriber are compared. Now in code, we give WARNING for missing sequences on publisher as well. Do we need to mention that here? IIUC, this WARNING for missing sequences can come only if the worker is respawned to sync unmatched/failed sequences and meanwhile any one of failed sequences is dropped on publisher. But it will be good to mention it briefly in doc. thanks Shveta
