Re: pgsql: Allow a partitioned table to have a default partition.

From: Thomas Munro <thomas(dot)munro(at)enterprisedb(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Robert Haas <rhaas(at)postgresql(dot)org>, pgsql-committers <pgsql-committers(at)postgresql(dot)org>
Subject: Re: pgsql: Allow a partitioned table to have a default partition.
Date: 2017-09-08 22:36:35
Message-ID: CAEepm=0bsTn90E8iGQ1FKOnyx=SsPw-nsOH6G_B4DthjnF0i7A@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-committers

On Sat, Sep 9, 2017 at 10:23 AM, Thomas Munro
<thomas(dot)munro(at)enterprisedb(dot)com> wrote:
> On Sat, Sep 9, 2017 at 9:49 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>> Robert Haas <rhaas(at)postgresql(dot)org> writes:
>>> Allow a partitioned table to have a default partition.
>>
>> Early buildfarm returns not good.
>>
>> It passes here, so the problem isn't obvious.
>
> map_partition_varattnos() doesn't always write to *found_whole_row, so
> it's uninitialised here in generate_partition_qual():
>
> result = map_partition_varattnos(result, 1, rel, parent,
> &found_whole_row);
> /* There can never be a whole-row reference here */
> if (found_whole_row)
> elog(ERROR, "unexpected whole-row reference found in partition key");

Hmm, no. I take that back.

--
Thomas Munro
http://www.enterprisedb.com

In response to

Responses

Browse pgsql-committers by date

  From Date Subject
Next Message Tom Lane 2017-09-08 22:50:26 Re: pgsql: Allow a partitioned table to have a default partition.
Previous Message Thomas Munro 2017-09-08 22:23:31 Re: pgsql: Allow a partitioned table to have a default partition.