Re: Commit 4dba331cb3 broke ATTACH PARTITION behaviour. - Mailing list pgsql-hackers

From Alvaro Herrera
Subject Re: Commit 4dba331cb3 broke ATTACH PARTITION behaviour.
Date
Msg-id 20180402160023.ic5ti6tyei3zjqar@alvherre.pgsql
Whole thread Raw
In response to Re: Commit 4dba331cb3 broke ATTACH PARTITION behaviour.  (Amit Langote <Langote_Amit_f8@lab.ntt.co.jp>)
Responses Re: Commit 4dba331cb3 broke ATTACH PARTITION behaviour.  (Alvaro Herrera <alvherre@alvh.no-ip.org>)
List pgsql-hackers
>      /*
> -     * Check whether default partition has a row that would fit the partition
> -     * being attached.
> +     * Check if the default partition contains a row that would belong in the
> +     * partition being attached.
>       */
> -    defaultPartOid =
> -        get_default_oid_from_partdesc(RelationGetPartitionDesc(rel));
>      if (OidIsValid(defaultPartOid))

Oh my.  This code is terrible, and I think this patch is wrong.  More
later.

-- 
Álvaro Herrera                https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services


pgsql-hackers by date:

Previous
From: Peter Eisentraut
Date:
Subject: Re: disable SSL compression?
Next
From: Dmitry Ivanov
Date:
Subject: Re: new function for tsquery creartion