| From: | Amit Khandekar <amitdkhan(dot)pg(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>, Rajkumar Raghuwanshi <rajkumar(dot)raghuwanshi(at)enterprisedb(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Robert Haas <robertmhaas(at)gmail(dot)com>, amul sul <sulamul(at)gmail(dot)com> |
| Subject: | Re: server crashed with TRAP: FailedAssertion("!(!parallel_aware || pathnode->path.parallel_safe)" |
| Date: | 2018-06-18 09:39:47 |
| Message-ID: | CAJ3gD9faV-v9Z+3NQ4gyEmveTeNRG1QWY+BSZhGkxq6t0Jk7CA@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On 16 June 2018 at 19:30, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
> On Sat, Jun 16, 2018 at 10:44 AM, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>> Yeah, or perhaps disallow creation of any partial paths at the first
>> place like in attached. This will save us some work as well.
>>
>
> Attached patch contains test case as well. I have tried to come up
> with some simple test, but couldn't come up with anything much simpler
> than reported by Rajkumar, so decided to use the test case provided by
> him.
>
Thanks for the patch. I had a look at it, and it looks good to me. One
minor comment :
+-- Parallel Append is not be used when the subpath depends on the outer param
"is not be used" => "is not to be used"
--
Thanks,
-Amit Khandekar
EnterpriseDB Corporation
The Postgres Database Company
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Amit Khandekar | 2018-06-18 09:41:18 | Re: server crashed with TRAP: FailedAssertion("!(!parallel_aware || pathnode->path.parallel_safe)" |
| Previous Message | Amit Khandekar | 2018-06-18 09:32:56 | Re: AtEOXact_ApplyLauncher() and subtransactions |