From: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
---|---|
To: | David Rowley <david(dot)rowley(at)2ndquadrant(dot)com> |
Cc: | Jaime Casanova <jaime(dot)casanova(at)2ndquadrant(dot)com>, Pg Hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: FailedAssertion on partprune |
Date: | 2018-07-24 13:46:03 |
Message-ID: | 984.1532439963@sss.pgh.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
David Rowley <david(dot)rowley(at)2ndquadrant(dot)com> writes:
> On 24 July 2018 at 16:55, Jaime Casanova <jaime(dot)casanova(at)2ndquadrant(dot)com> wrote:
>> I was trying sqlsmith on REL_11_STABLE (commit
>> 1b957e59b92dc44c14708762f882d7910463a9ac) with a database i have at
>> hand, and got an assertion failure.
> In this case, the 2nd to 5th Append subplan's parents are actually
> partitioned tables rather than leaf partitions.
> make_partition_pruneinfo() assumes these will always be leaf
> partitions. Looking more closely at accumulate_append_subpath() I can
> see that there are cases where it won't pull sub-Append paths into the
> main Append. That's what we're seeing in this plan.
Hm, wouldn't this be fixed by your pending patch at
<CAKJS1f_eYwHk2x0xX7qW42rV_GRsJGBMe3AqN9MYLRSs1S+CiA(at)mail(dot)gmail(dot)com>
?
(I apologize for having seemingly dropped the ball on that thread;
RL has intruded on my time a bit lately. I hope to get it dealt with
next week.)
regards, tom lane
From | Date | Subject | |
---|---|---|---|
Next Message | Daniel Verite | 2018-07-24 13:58:45 | Re: Stored procedures and out parameters |
Previous Message | Haribabu Kommi | 2018-07-24 13:31:51 | Re: Pluggable Storage - Andres's take |