Hi, On Wed, Oct 14, 2026 at 12:26 PM shveta malik <[email protected]> wrote: > > On Wed, Oct 14, 2026 at 7:43 AM Zhijie Hou <[email protected]> wrote: > > I also re-confirmed locally that all the parameters you reported (e.g. > > DateStyle, IntervalStyle, extra_float_digits) have similar issues, and > > setting > > the GUCs manually when building the CLT row is an appropriate way to fix > > this. > > Thanks for verifying. > > > Please see the attachment for a TAP test on top of v81 that tries to > > reproduce > > all of them. And if we plan to fix this along with the main patch, I think > > it > > would be worth keeping a simple test case like the attached one. > > > > I like the idea of test-patch. But the test has a few problems. It > did not compile and run on my machine. Problems were: > > 1) > [09:09:53] t/035_conflicts.pl .. Global symbol "$node_A" requires > explicit package name (did you forget to declare "my $node_A"?) at > t/035_conflicts.pl line 209. > Execution of t/035_conflicts.pl aborted due to compilation errors. > > Problem was this part: > +# Wait for the apply worker to stop > +$node_A->poll_query_until('postgres', > + "SELECT count(*) = 0 FROM pg_stat_activity WHERE backend_type = > 'logical replication apply worker'" > > I think we need to change node_A to node_subscriber. Otherwise also, > it was wrong as node_A is assigned as: > > my $node_A = $node_publisher > > And we want to stop worker on subscriber. > > > 2) > After fixing above, the testcase fails for me: > > [09:12:35] t/035_conflicts.pl .. 6/? > # Failed test 'CLT JSON stores timestamp in ISO format' > # at t/035_conflicts.pl line 239. > # '2026-03-04 10:11:12.123456+05:30' > # doesn't match '(?^:^\d{4}-\d{2}-\d{2} > \d{2}:\d{2}:\d{2}(\.\d+)?[+-]\d{2}$)' > > This is because it could not match timezone offsets that include > minutes (e.g. +05:30 in IST) on my machine. This needed to be changed > to: > qr/^\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2}(\.\d+)?[+-]\d{2}(:\d{2}){0,2}$/, > > ~~
Oh, right. I missed to merge some local changes in the patch. > > I have addressed these 2 test-issues in the attached patch. > > Also added code to fix intervalstyle and extra_float issues. Please review > once. Thanks for updating the patch, it looks good to me. Best Regards, Zhijie Hou
