On Fri, Nov 28, 2025 at 6:06 AM Peter Smith <[email protected]> wrote: > > Some review comments for v8-0001.
Thank Peter, yes these all make sense and will fix in next version along with other comments by Vignesh/Shveta and Amit, except one comment > 9. > +-- ok - conflict_log_table should not be published with ALL TABLE > +CREATE PUBLICATION pub FOR TABLES IN SCHEMA clt; > +SELECT * FROM pg_publication_tables WHERE pubname = 'pub'; > + pubname | schemaname | tablename | attnames | rowfilter > +---------+------------+-----------+----------+----------- > +(0 rows) > > Perhaps you should repeat this same test but using FOR ALL TABLES, > instead of only FOR TABLES IN SCHEMA I will have to see how we can safely do this in testing without having any side effects on the concurrent test, generally we run publication.sql and subscription.sql concurrently in regression test so if we do FOR ALL TABLES it can affect each others, one option is to don't run these 2 test concurrently, I think we can do that as there is no real concurrency we are testing by running them concurrently, any thought on this? -- Regards, Dilip Kumar Google
