Thread

Re: Fix SPLIT PARTITION bound-overlap bug and other improvements

Alexander Korotkov <aekorotkov@gmail.com> — 2026-05-17T21:45:17Z

Hi, Chao!

On Thu, May 14, 2026 at 9:59 AM Chao Li <li.evan.chao@gmail.com> wrote:
> To make this patch easier to process, I split it into 4 commits:
>
> 0001 - Fixes the bound-overlap bug
> 0002 - Fix the incorrect HINT message for the DEFAULT case
> 0003 - Fix the incorrect description about combined bound in the SGML doc
> 0004 - Reject only-create-default-partition usage

Thank you for your work.  I've revised the patchset.
0002 - I've also fixed gramma of hints in other branches
0004 - In the check_split_partition_not_same_bound(), calling
partition_bounds_create() and partition_bounds_equal() looks a bit
heavyweight.  It doesn't matter much performance-wise, but it feels
like start processing from scratch while we're on quite late stage
already.  I've replaced that with more lightweight check.  Also I
removed dealing with memory context.  This code implies small
non-repetitive memory allocations which only lives during DDL
operation, no need to wrap them with memory context as we don't do so
in other places.

Any objections if I commit this?

------
Regards,
Alexander Korotkov
Supabase