Re: [HACKERS] GatherMerge misses to push target list

Поиск
Список
Период
Сортировка
От Robert Haas
Тема Re: [HACKERS] GatherMerge misses to push target list
Дата
Msg-id CA+TgmoY6ZYfmJUKwhJ1wFuFk2yXZj8OJrNHRskcMXvt_UTBNVA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [HACKERS] GatherMerge misses to push target list  (Rushabh Lathia <rushabh.lathia@gmail.com>)
Ответы Re: [HACKERS] GatherMerge misses to push target list
Список pgsql-hackers
On Mon, Sep 18, 2017 at 7:02 AM, Rushabh Lathia
<rushabh.lathia@gmail.com> wrote:
>> In that case, can you please mark the patch [1] as ready for committer in
>> CF app
>
> Done.

I think this patch is mostly correct, but I think the change to
planner.c isn't quite right.  ordered_rel->reltarget is just a dummy
target list that produces nothing.  Instead, I think we should pass
path->pathtarget, representing the idea that whatever Gather Merge
produces as output is the same as what you put into it.

See attached.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Вложения

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

Предыдущее
От: Graham Leggett
Дата:
Сообщение: Re: [HACKERS] [Patch] Log SSL certificate verification errors
Следующее
От: Robert Haas
Дата:
Сообщение: Re: [HACKERS] [PROPOSAL] Temporal query processing with range types