On Tue, May 20, 2025 at 11:13 AM Peter Smith <smithpb2...@gmail.com> wrote:
>
> > Test-scenario:
> > --Created 250 sequences on both pub and sub.
> > --There were 10 sequences mismatched.
> > --Sequence replication worked as expected. Logs look better now:
> >
> > LOG: Logical replication sequence synchronization for subscription
> > "sub1" - total unsynchronized: 250; batch #1 = 100 attempted, 97
> > succeeded, 3 mismatched
> > LOG: Logical replication sequence synchronization for subscription
> > "sub1" - total unsynchronized: 250; batch #2 = 100 attempted, 95
> > succeeded, 5 mismatched
> > LOG: Logical replication sequence synchronization for subscription
> > "sub1" - total unsynchronized: 250; batch #3 = 50 attempted, 48
> > succeeded, 2 mismatched
> >
>
> When there are many batches required, it seems a bit strange to repeat
> the same "total unsynchronized" over and over.
>
> Would it be better to show the total number once, and thereafter show
> the number of sequences remaining to be processed as they tick down?
>
> e.g.
> LOG: Logical replication sequence synchronization for subscription
> "sub1" - total unsynchronized = 250
> LOG: Logical replication sequence synchronization for subscription
> "sub1" - batch #1 = 100 attempted, 97 succeeded, 3 mismatched, 150
> remaining
> LOG: Logical replication sequence synchronization for subscription
> "sub1" - batch #2 = 100 attempted, 95 succeeded, 5 mismatched, 50
> remaining
> LOG: Logical replication sequence synchronization for subscription
> "sub1" - batch #3 = 50 attempted, 48 succeeded, 2 mismatched, 0
> remaining
+1 on log change suggestions.
Please find few more comments:
1)
Temporary sequences will not be replicated, shall we mention this in
doc under '29.7. Replicating Sequences'?
2)
CREATE publication pub1 for all sequences WITH (publish = 'insert,
update, truncate');
I think it does not make sense to give 'publish' as above (or
publish_via_partition_root) for 'all sequences' publication. Shall we
display a WARNING that such will be ignored for 'all sequences' and
let the create-publication go ahead? Thoughts? Also the doc for
publish* option in the CREATE-PUBLICATION page needs to specify that
these options are not-applicable for ALL SEQUENCES publication.
3)
It will be good to move create_publication.sgml as well to the last
patch where all other doc changes are present. I was trying to find
this change in the last patch but ultimately found it in pacth002.
4)
Currently the log is:
------
LOG: logical replication sequence synchronization worker for
subscription "sub1" has started
LOG: Logical replication sequence synchronization for subscription
"sub1" - total unsynchronized: 1; batch #1 = 1 attempted, 0 succeeded,
1 mismatched
WARNING: parameters differ for the remote and local sequences
("public.myseq34") for subscription "sub1"
HINT: Alter/Re-create local sequences to have the same parameters as
the remote sequences.
WARNING: sequence synchronization worker failed: one or more
sequences have mismatched parameters between the publisher and
subscriber
LOG: logical replication sequence synchronization worker for
subscription "sub1" has finished
-----
Do we need both?
--WARNING: sequence synchronization worker failed.
--LOG: logical replication sequence synchronization worker for
subscription "sub1" has finished
This WARNING repeats previously stated information. I feel we can get
rid of it, unless there is a chance of some new error which we are
trying to display in this WARNING other than mismatched seq error?
thanks
Shveta
