| 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
| 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. |