From: | amul sul <sulamul(at)gmail(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Amit Khandekar <amitdkhan(dot)pg(at)gmail(dot)com>, Rafia Sabih <rafia(dot)sabih(at)enterprisedb(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Andres Freund <andres(at)anarazel(dot)de>, Ashutosh Bapat <ashutosh(dot)bapat(at)enterprisedb(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: [HACKERS] Parallel Append implementation |
Date: | 2017-11-23 04:15:08 |
Message-ID: | CAAJ_b964Ww8CW0zS+=f0jzz_rGfK5-AdeVCt+BSj1KHPz3csZg@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Nov 22, 2017 at 1:44 AM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Tue, Nov 21, 2017 at 6:57 AM, amul sul <sulamul(at)gmail(dot)com> wrote:
>> By doing following change on the v19 patch does the fix for me:
>>
>> --- a/src/backend/executor/nodeAppend.c
>> +++ b/src/backend/executor/nodeAppend.c
>> @@ -489,11 +489,9 @@ choose_next_subplan_for_worker(AppendState *node)
>> }
>>
>> /* Pick the plan we found, and advance pa_next_plan one more time. */
>> - node->as_whichplan = pstate->pa_next_plan;
>> + node->as_whichplan = pstate->pa_next_plan++;
>> if (pstate->pa_next_plan == node->as_nplans)
>> pstate->pa_next_plan = append->first_partial_plan;
>> - else
>> - pstate->pa_next_plan++;
>>
>> /* If non-partial, immediately mark as finished. */
>> if (node->as_whichplan < append->first_partial_plan)
>>
>> Attaching patch does same changes to Amit's ParallelAppend_v19_rebased.patch.
>
> Yes, that looks like a correct fix. Thanks.
>
Attaching updated version of "ParallelAppend_v19_rebased" includes this fix.
Regards,
Amul
Attachment | Content-Type | Size |
---|---|---|
ParallelAppend_v20.patch | application/octet-stream | 60.0 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Craig Ringer | 2017-11-23 05:59:46 | Re: [HACKERS] SQL procedures |
Previous Message | Tom Lane | 2017-11-23 03:58:05 | Re: has_sequence_privilege() never got the memo |