From: | Rajkumar Raghuwanshi <rajkumar(dot)raghuwanshi(at)enterprisedb(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Amit Khandekar <amitdkhan(dot)pg(at)gmail(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, amul sul <sulamul(at)gmail(dot)com> |
Subject: | Re: server crashed with TRAP: FailedAssertion("!(!parallel_aware || pathnode->path.parallel_safe)" |
Date: | 2018-06-22 06:28:04 |
Message-ID: | CAKcux6k9=kEik_St6Dk_fQ5Wu-EGSgi6p7fxmtqSTTNqgv6SXw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Thanks for commit. I have verified reported case. it is fixed now.
Thanks & Regards,
Rajkumar Raghuwanshi
QMG, EnterpriseDB Corporation
On Thu, Jun 21, 2018 at 7:21 PM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
> Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> writes:
> > On Thu, Jun 21, 2018 at 11:51 AM, Amit Khandekar <amitdkhan(dot)pg(at)gmail(dot)com>
> wrote:
> >> After list_concat, the subpaths no longer has only non-partial paths,
> >> which it is supposed to have. So it anyways should not be used in the
> >> subsequent code in that function. So I think the following change
> >> should be good.
> >> - foreach(l, subpaths)
> >> + foreach(l, pathnode->subpaths)
>
> Patch LGTM.
>
> > Thanks, Tom, would you like to go-ahead and commit this change as this
> > is suggested by you or would you like me to take care of this or do
> > you want to wait for Robert's input?
>
> Please push --- I'm busy getting ready to leave for vacation ...
>
> regards, tom lane
>
From | Date | Subject | |
---|---|---|---|
Next Message | David Rowley | 2018-06-22 06:28:08 | Speeding up INSERTs and UPDATEs to partitioned tables |
Previous Message | Rajkumar Raghuwanshi | 2018-06-22 06:26:21 | Re: Server crashed with TRAP: FailedAssertion("!(parallel_workers > 0)" when partitionwise_aggregate true. |