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.
Cheers
