From: | Dilip Kumar <dilipbalaut(at)gmail(dot)com> |
---|---|
To: | "Hou, Zhijie" <houzj(dot)fnst(at)cn(dot)fujitsu(dot)com> |
Cc: | Bharath Rupireddy <bharath(dot)rupireddyforpostgres(at)gmail(dot)com>, Amit Kapila <amit(dot)kapila16(at)gmail(dot)com>, Luc Vlaming <luc(at)swarm64(dot)com>, PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Zhihong Yu <zyu(at)yugabyte(dot)com> |
Subject: | Re: Parallel Inserts in CREATE TABLE AS |
Date: | 2020-12-10 11:18:55 |
Message-ID: | CAFiTN-sQz15EKH_UgvHdgsjzP3xyRt668H0=xU0-e+4EP5-CAA@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Dec 10, 2020 at 3:59 PM Hou, Zhijie <houzj(dot)fnst(at)cn(dot)fujitsu(dot)com> wrote:
>
> Hi
>
> + allow = ps && IsA(ps, GatherState) && !ps->ps_ProjInfo &&
> + plannedstmt->parallelModeNeeded &&
> + plannedstmt->planTree &&
> + IsA(plannedstmt->planTree, Gather) &&
> + plannedstmt->planTree->lefttree &&
> + plannedstmt->planTree->lefttree->parallel_aware &&
> + plannedstmt->planTree->lefttree->parallel_safe;
>
> I noticed it check both IsA(ps, GatherState) and IsA(plannedstmt->planTree, Gather).
> Does it mean it is possible that IsA(ps, GatherState) is true but IsA(plannedstmt->planTree, Gather) is false ?
>
> I did some test but did not find a case like that.
>
This seems like an extra check. Apart from that if we combine 0001
and 0002 there should be an additional protection so that it should
not happen that in cost_gather we have ignored the parallel tuple cost
and now we are rejecting the parallel insert. Probably we should add
an assert.
--
Regards,
Dilip Kumar
EnterpriseDB: http://www.enterprisedb.com
From | Date | Subject | |
---|---|---|---|
Next Message | Neil Chen | 2020-12-10 11:26:48 | Re: Proposed patch for key managment |
Previous Message | Amit Kapila | 2020-12-10 11:12:12 | Re: [Patch] Optimize dropping of relation buffers using dlist |