| From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
|---|---|
| To: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
| Cc: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Andreas Seltenreich <seltenreich(at)gmx(dot)de>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: [sqlsmith] Failed assertion in parallel worker (ExecInitSubPlan) |
| Date: | 2016-06-03 18:31:57 |
| Message-ID: | CA+TgmoZGRDqBLoVx8fbCwLZMK=W3ZbBetjg8TQ19Xaip2SNgbw@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Sat, May 7, 2016 at 9:07 AM, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> From the above output it is clear that parallel restricted function is
> pushed down below gather node. I found that though we have have care fully
> avoided to push pathtarget below GatherPath in apply_projection_to_path() if
> pathtarget contains any parallel unsafe or parallel restricted clause, but
> we are separately also trying to apply pathtarget to partialpath list which
> doesn't seem to be the correct way even if it is required. I think this has
> been added during parallel aggregate patch and it seems to me this is not
> required after the changes related to GatherPath in
> apply_projection_to_path().
Good catch. Committed.
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
| From | Date | Subject | |
|---|---|---|---|
| Next Message | David G. Johnston | 2016-06-03 18:42:17 | Re: PostmasterPid not marked with PGDLLIMPORT |
| Previous Message | Merlin Moncure | 2016-06-03 18:22:56 | Re: Changed SRF in targetlist handling |