From: | Rahila Syed <rahilasyed90(at)gmail(dot)com> |
---|---|
To: | amul sul <sulamul(at)gmail(dot)com> |
Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Keith Fiske <keith(at)omniti(dot)com>, Amit Langote <Langote_Amit_f8(at)lab(dot)ntt(dot)co(dot)jp>, Rushabh Lathia <rushabh(dot)lathia(at)gmail(dot)com>, David Steele <david(at)pgmasters(dot)net>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: Adding support for Default partition in partitioning |
Date: | 2017-05-04 11:44:02 |
Message-ID: | CAH2L28vkHgFg+3bPMdYmW4DbkL4TKhyi5FrCeNjupdpFBq4SdA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Hello Amul,
Thanks for reporting. Please find attached an updated patch which fixes the
above.
Also, the attached patch includes changes in syntax proposed upthread.
The syntax implemented in this patch is as follows,
CREATE TABLE p11 PARTITION OF p1 DEFAULT;
Thank you,
Rahila Syed
On Thu, May 4, 2017 at 4:02 PM, amul sul <sulamul(at)gmail(dot)com> wrote:
> On Tue, May 2, 2017 at 9:33 PM, Rahila Syed <rahilasyed90(at)gmail(dot)com>
> wrote:
> > Please find attached updated patch with review comments by Robert and
> Jeevan
> > implemented.
> >
> Patch v8 got clean apply on latest head but server got crash at data
> insert in the following test:
>
> -- Create test table
> CREATE TABLE test ( a int, b date) PARTITION BY LIST (a);
> CREATE TABLE p1 PARTITION OF test FOR VALUES IN (DEFAULT) PARTITION BY
> LIST(b);
> CREATE TABLE p11 PARTITION OF p1 FOR VALUES IN (DEFAULT);
>
> -- crash
> INSERT INTO test VALUES (210,'1/1/2002');
>
> Regards,
> Amul
>
Attachment | Content-Type | Size |
---|---|---|
default_partition_v9.patch | application/x-download | 22.5 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Julien Rouhaud | 2017-05-04 11:45:01 | Re: CTE inlining |
Previous Message | Heikki Linnakangas | 2017-05-04 11:37:15 | Re: password_encryption, default and 'plain' support |