On Tue, Sep 7, 2021 at 11:06 AM Zhihong Yu <z...@yugabyte.com> wrote: > > > > On Tue, Sep 7, 2021 at 6:17 AM James Coleman <jtc...@gmail.com> wrote: >> >> On Wed, Sep 1, 2021 at 7:06 AM Daniel Gustafsson <dan...@yesql.se> wrote: >> > >> > > On 7 May 2021, at 18:30, James Coleman <jtc...@gmail.com> wrote: >> > >> > > ..here we are now, and I finally have this patch cleaned up >> > > enough to share. >> > >> > This patch no longer applies to HEAD, can you please submit a rebased >> > version? >> >> See attached. >> >> Thanks, >> James > > Hi, > For v2-0002-Parallel-query-support-for-basic-correlated-subqu.patch : > > + * is when we're going to execute multiple partial parths in parallel > > parths -> paths > > if (index->amcanparallel && > - rel->consider_parallel && outer_relids == NULL && > - scantype != ST_BITMAPSCAN) > + rel->consider_parallel && outer_relids == NULL && > + scantype != ST_BITMAPSCAN) > > the change above seems unnecessary since the first line of if condition > doesn't change. > Similar comment for the next hunk. > > + * It's not a partial path; it'a a full path that is executed as > a subquery. > > it'a a -> it's a > > + /* rel->consider_parallel_rechecking_params = false; */ > + /* rel->partial_pathlist = NIL; */ > > The commented code can be taken out.
Thanks for taking a look at this. See updated patch series attached. James Coleman
v3-0001-Allow-parallel-LATERAL-subqueries-with-LIMIT-OFFS.patch
Description: Binary data
v3-0002-Parallel-query-support-for-basic-correlated-subqu.patch
Description: Binary data
v3-0003-Other-places-to-consider-for-completeness.patch
Description: Binary data