Re: Adding support for Default partition in partitioning

From: Rahila Syed <rahilasyed90(at)gmail(dot)com>
To: David Steele <david(at)pgmasters(dot)net>
Cc: Rushabh Lathia <rushabh(dot)lathia(at)gmail(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, 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-04-04 13:30:21
Views: Raw Message | Whole Thread | Download mbox
Lists: pgsql-hackers


Please find attached an updated patch.
Following has been accomplished in this update:

1. A new partition can be added after default partition if there are no
conflicting rows in default partition.
2. Solved the crash reported earlier.

Thank you,
Rahila Syed

On Tue, Apr 4, 2017 at 6:22 PM, David Steele <david(at)pgmasters(dot)net> wrote:

> On 3/31/17 10:45 AM, David Steele wrote:
> > On 3/29/17 8:13 AM, Rahila Syed wrote:
> >
> >> Thanks for reporting. I have identified the problem and have a fix.
> >> Currently working on allowing
> >> adding a partition after default partition if the default partition does
> >> not have any conflicting rows.
> >> Will update the patch with both of these.
> >
> > The CF has been extended but until April 7 but time is still growing
> > short. Please respond with a new patch by 2017-04-04 00:00 AoE (UTC-12)
> > or this submission will be marked "Returned with Feedback".
> This submission has been marked "Returned with Feedback". Please feel
> free to resubmit to a future commitfest.
> Regards,
> --
> -David
> david(at)pgmasters(dot)net

Attachment Content-Type Size
default_partition_v4.patch application/x-download 19.2 KB

In response to


Browse pgsql-hackers by date

  From Date Subject
Next Message David Steele 2017-04-04 13:32:24 Re: Re: PATCH: pageinspect / add page_checksum and bt_page_items(bytea)
Previous Message Peter Eisentraut 2017-04-04 13:28:34 Re: [COMMITTERS] pgsql: Add COMMENT and SECURITY LABEL support for publications and subs