Re: Adding support for Default partition in partitioning
Jeevan Ladhe <jeevan.ladhe@enterprisedb.com>
From: Jeevan Ladhe <jeevan.ladhe@enterprisedb.com>
To: Rajkumar Raghuwanshi <rajkumar.raghuwanshi@enterprisedb.com>
Cc: Rahila Syed <rahilasyed90@gmail.com>, amul sul <sulamul@gmail.com>, Robert Haas <robertmhaas@gmail.com>, Keith Fiske <keith@omniti.com>, Amit Langote <Langote_Amit_f8@lab.ntt.co.jp>,
Rushabh Lathia <rushabh.lathia@gmail.com>, David Steele <david@pgmasters.net>,
Peter Eisentraut <peter.eisentraut@2ndquadrant.com>, PostgreSQL-development <pgsql-hackers@postgresql.org>
Date: 2017-05-04T20:28:19Z
Lists: pgsql-hackers
While reviewing the code I was trying to explore more cases, and I here comes an open question to my mind: should we allow the default partition table to be partitioned further? If we allow it(as in the current case) then observe following case, where I have defined a default partitioned which is further partitioned on a different column. postgres=# CREATE TABLE test ( a int, b int, c int) PARTITION BY LIST (a); CREATE TABLE postgres=# CREATE TABLE test_p1 PARTITION OF test FOR VALUES IN(4, 5, 6, 7, 8); CREATE TABLE postgres=# CREATE TABLE test_pd PARTITION OF test DEFAULT PARTITION BY LIST(b); CREATE TABLE postgres=# INSERT INTO test VALUES (20, 24, 12); ERROR: no partition of relation "test_pd" found for row DETAIL: Partition key of the failing row contains (b) = (24). Note, that it does not allow inserting the tuple(20, 24, 12) because though a=20 would fall in default partition i.e. test_pd, table test_pd itself is further partitioned and does not have any partition satisfying b=24. Further if I define a default partition for table test_pd, the the tuple gets inserted. Doesn't this sound like the whole purpose of having DEFAULT partition on test table is defeated? Any views? Regards, Jeevan Ladhe
Commits
-
Allow a partitioned table to have a default partition.
- 6f6b99d1335b 11.0 landed
-
Adjust min/max values when changing sequence type
- 60a0b2ec8943 10.0 cited
-
BRIN auto-summarization
- 7526e10224f0 10.0 cited