| From: | Alexander Korotkov <akorotkov(at)postgresql(dot)org> |
|---|---|
| To: | pgsql-committers(at)lists(dot)postgresql(dot)org |
| Subject: | pgsql: Reject degenerate SPLIT PARTITION with DEFAULT partition |
| Date: | 2026-05-25 08:59:35 |
| Message-ID: | E1wRR9r-0011yZ-26@gemulon.postgresql.org |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-committers |
Reject degenerate SPLIT PARTITION with DEFAULT partition
ALTER TABLE ... SPLIT PARTITION allows a DEFAULT partition to be created
as one of the replacement partitions when the parent table does not
already have one. However, it should not allow the degenerate case where
a non-DEFAULT partition keeps exactly the same bound as the split
partition and the command merely adds a DEFAULT partition through the
SPLIT PARTITION path.
Detect that case by comparing the bound of the split partition with the
bound of the only non-DEFAULT replacement partition, and raise an error
when they are the same. Users should add a DEFAULT partition directly
with CREATE TABLE ... PARTITION OF ... DEFAULT or ALTER TABLE ... ATTACH
PARTITION ... DEFAULT instead.
The comparison goes through the partition operator family rather than
byte equality so that values which are binary-different but compare
equal under the partition key's comparator are treated as the same
bound. The corresponding regression test uses a float8 LIST partition
with -0.0 and 0.0 -- they have different bit patterns but are equal
under float8 -- to verify that a datumIsEqual()-based check would let
the degenerate split through while the partsupfunc-based check
correctly rejects it.
Author: Chao Li <lic(at)highgo(dot)com>
Reviewed-by: Alexander Korotkov <aekorotkov(at)gmail(dot)com>
Discussion: https://postgr.es/m/C18878AB-DEB2-4A61-9995-A035DD644B81@gmail.com
Branch
------
master
Details
-------
https://git.postgresql.org/pg/commitdiff/e64a9ba2b4fcd8956369ac6e2146ac816e590df2
Modified Files
--------------
src/backend/partitioning/partbounds.c | 150 ++++++++++++++++++++++++++
src/test/regress/expected/partition_split.out | 58 ++++++++++
src/test/regress/sql/partition_split.sql | 51 +++++++++
3 files changed, 259 insertions(+)
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Daniel Gustafsson | 2026-05-25 10:22:58 | pgsql: psql: Tab completion for CHECKPOINT FLUSH_UNLOGGED boolean optio |
| Previous Message | Michael Paquier | 2026-05-25 05:41:16 | pgsql: Fix size check in statext_dependencies_deserialize() |