On Tue, Jun 7, 2022, at 1:10 AM, Peter Smith wrote: > The logical replication tablesync ignores the publication 'publish' > operations during the initial data copy. > > This is current/known PG behaviour (e.g. as recently mentioned [1]) > but it was not documented anywhere. initial data synchronization != replication. publish parameter is a replication property; it is not a initial data synchronization property. Maybe we should make it clear like you are suggesting.
> This patch just documents the existing behaviour and gives some examples. Why did you add this information to that specific paragraph? IMO it belongs to a separate paragraph; I would add it as the first paragraph in that subsection. I suggest the following paragraph: <para> The initial data synchronization does not take into account the <literal>publish</literal> parameter to copy the existing data. </para> There is no point to link the Initial Snapshot subsection. That subsection is explaining the initial copy steps and you want to inform about the effect of a publication parameter on the initial copy. Although both are talking about the same topic (initial copy), that link to Initial Snapshot subsection won't add additional information about the publish parameter. You could expand the suggested sentence to make it clear what publish parameter is or even add a link to the CREATE PUBLICATION synopsis (that explains about publish parameter). You add an empty paragraph. Remove it. I'm not sure it deserves an example. It is an easy-to-understand concept and a good description is better than ~ 80 new lines. -- Euler Taveira EDB https://www.enterprisedb.com/
