| From: | Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> |
|---|---|
| To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
| Cc: | PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
| Subject: | Re: Observations in Parallel Append |
| Date: | 2018-01-04 03:54:41 |
| Message-ID: | CAA4eK1LHt=QF-t6+60nzEztcSZBpZA6sowRC3ZavUYpyd+1CCQ@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Thu, Jan 4, 2018 at 9:06 AM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> On Tue, Jan 2, 2018 at 11:11 PM, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com> wrote:
>> Attached, please find the patch which fixes this issue (Thanks to
>> Dilip for helping me in identifying the above case and fix). I have
>> also modified the comment atop function
>> choose_next_subplan_for_worker() as discussed above. The change to
>> remove unnecessary inclusion of spin.h is still in a separate patch as
>> posted above.
>
> Here's a combined patch with some cosmetic changes which I will commit
> if it looks OK to you.
>
Looks good to me.
--
With Regards,
Amit Kapila.
EnterpriseDB: http://www.enterprisedb.com
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Jing Wang | 2018-01-04 04:20:52 | Libpq support to connect to standby server as priority |
| Previous Message | Robert Haas | 2018-01-04 03:36:01 | Re: Observations in Parallel Append |