Re: [HACKERS] Partition-wise aggregation/grouping

Поиск
Список
Период
Сортировка
От Rajkumar Raghuwanshi
Тема Re: [HACKERS] Partition-wise aggregation/grouping
Дата
Msg-id CAKcux6=D4g846mNg=7kgeSTY_0TLSW=C5d-V1pZji7A5Oo7FGA@mail.gmail.com
обсуждение исходный текст
Ответ на Re: [HACKERS] Partition-wise aggregation/grouping  (Jeevan Chalke <jeevan.chalke@enterprisedb.com>)
Список pgsql-hackers


On Mon, Sep 18, 2017 at 12:37 PM, Jeevan Chalke <jeevan.chalke@enterprisedb.com> wrote:


On Tue, Sep 12, 2017 at 6:21 PM, Jeevan Chalke <jeevan.chalke@enterprisedb.com> wrote:


On Tue, Sep 12, 2017 at 3:24 PM, Rajkumar Raghuwanshi <rajkumar.raghuwanshi@enterprisedb.com> wrote:

Hi Jeevan,

I have started testing partition-wise-aggregate and got one observation, please take a look.
with the v2 patch, here if I change target list order, query is not picking full partition-wise-aggregate.

Thanks Rajkumar for reporting this.

I am looking into this issue and will post updated patch with the fix.

Logic for checking whether partition keys lead group by keys needs to be
updated here. The group by expressions can appear in any order without
affecting the final result. And thus, the need for partition keys should
be leading the group by keys to have full aggregation is not mandatory.
Instead we must ensure that the partition keys are part of the group by
keys to compute full aggregation on a partition.

Attached, revised patch-set with above fix.

Also, in test-cases, I have removed DROP/ANALYZE commands on child
relations and also removed VERBOSE from the EXPLAIN.

Notes:
HEAD: 8edacab209957520423770851351ab4013cb0167
Partition-wise Join patch-set version: v32

Thanks for the patch. I have tested it and issue is fixed now.  

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

Предыдущее
От: Rafal Pietrak
Дата:
Сообщение: [HACKERS] global indices
Следующее
От: Amit Kapila
Дата:
Сообщение: Re: [HACKERS] GatherMerge misses to push target list