Hi,

On Thursday, August 20, 2026 8:46 PM Alexander Korotkov <[email protected]> 
wrote:
> 
> Added as 0005 patch to the patchset.
> 

I took a look at 0001 and 0002.

0001 looks OK to me.

For 0002, I think we should also disallow the command when the partition is
explicitly listed in the publication's EXCEPT TABLE list. Otherwise, changes on
partitions that were previously ignored would start being replicated after
splitting (or merging), which could be unexpected.

BTW, this patch also disallows SPLIT/MERGE when both the parent and child tables
are explicitly added to a publication. That case is actually safe, since
publishing the parent already covers the partition. But if the intent is to keep
the check simple and avoid adding complexity for this infrequent case, I think
that's acceptable - though it might be worth adding a comment to explain the
reasoning.

Best Regards,
Zhijie Hou

Reply via email to