On Tue, Oct 14, 2025 at 5:08 PM Dilip Kumar <[email protected]> wrote: > > On Tue, Oct 14, 2025 at 3:36 PM Amit Kapila <[email protected]> wrote: > > 0001 and 0002 looks good, >
Thanks, I pushed 0001. I feel it is better to next commit refactoring patch v20251014-0003-Reorganize-tablesync-Code-and-Introduce-sy as that would be less controversial. What do you think? > except this duplicate version checking code below in fetch_relation_list > [1][2], I mean check_relkind and sequence fetching both are related changes > and start from version 19, so we can do a single check. Instead of the > 'check_relkind' variable name we can change it to 'support_relkind_seq' or > something like that and then we can use this in both checks. > Sounds reasonable to me. -- With Regards, Amit Kapila.
