Re: Parallelize correlated subqueries that execute within each worker

Поиск
Список
Период
Сортировка
От Zhihong Yu
Тема Re: Parallelize correlated subqueries that execute within each worker
Дата
Msg-id CALNJ-vTpbZtTcJ+Y-kUc5D5_rNr4-1YY=-T2WF+t5425MSuCtQ@mail.gmail.com
обсуждение исходный текст
Ответ на Re: Parallelize correlated subqueries that execute within each worker  (James Coleman <jtc331@gmail.com>)
Ответы Re: Parallelize correlated subqueries that execute within each worker  (James Coleman <jtc331@gmail.com>)
Список pgsql-hackers
Дерево обсуждения


On Tue, Sep 7, 2021 at 6:17 AM James Coleman <jtc331@gmail.com> wrote:
On Wed, Sep 1, 2021 at 7:06 AM Daniel Gustafsson <daniel@yesql.se> wrote:
>
> > On 7 May 2021, at 18:30, James Coleman <jtc331@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

В списке pgsql-hackers по дате отправления:

Предыдущее
От: Aleksander Alekseev
Дата:
Сообщение: Re: [BUG?] SET TIME ZONE doesn't work with abbreviations
Следующее
От: Tomas Vondra
Дата:
Сообщение: Re: Gather performance analysis