Comment about set_join_pathlist_hook()

Поиск
Список
Период
Сортировка
От Etsuro Fujita
Тема Comment about set_join_pathlist_hook()
Дата
Msg-id CAPmGK15SBPA1nr3Aqsdm+YyS-ay0Ayo2BRYQ8_A2To9eLqwopQ@mail.gmail.com
обсуждение исходный текст
Ответы Re: Comment about set_join_pathlist_hook()  (David Rowley <dgrowleyml@gmail.com>)
Re: Comment about set_join_pathlist_hook()  ("Lepikhov Andrei" <a.lepikhov@postgrespro.ru>)
Список pgsql-hackers
Hi,

What I am concerned about from the report [1] is that this comment is
a bit too terse; it might cause a misunderstanding that extensions can
do different things than we intend to allow:

    /*
     * 6. Finally, give extensions a chance to manipulate the path list.
     */
    if (set_join_pathlist_hook)
        set_join_pathlist_hook(root, joinrel, outerrel, innerrel,
                               jointype, &extra);

So I would like to propose to extend the comment to explain what they
can do, as in the comment about set_rel_pathlist_hook() in allpaths.c.
Attached is a patch for that.

Best regards,
Etsuro Fujita

[1] https://www.postgresql.org/message-id/CACawEhV%3D%2BQ0HXrcDergbTR9EkVFukgRPMTZbRFL-YK5CRmvYag%40mail.gmail.com

Вложения

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

Предыдущее
От: Yuya Watari
Дата:
Сообщение: Re: [PoC] Reducing planning time when tables have many partitions
Следующее
От: Amit Kapila
Дата:
Сообщение: Re: [PoC] pg_upgrade: allow to upgrade publisher node