Re: WIP Patch for GROUPING SETS phase 1

Поиск
Список
Период
Сортировка
От Andrew Gierth
Тема Re: WIP Patch for GROUPING SETS phase 1
Дата
Msg-id 87lhps69jg.fsf@news-spur.riddles.org.uk
обсуждение исходный текст
Ответ на Re: WIP Patch for GROUPING SETS phase 1  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers
>>>>> "Tom" == Tom Lane <tgl@sss.pgh.pa.us> writes:
>>> Honestly, ChainAggregate is _trivial_ compared to trying to make the>>> GroupAggregate code deal with multiple
inputs,or trying to make some>>> new sort of plumbing node to feed input to those sorts.  (You'd think>>> that it
shouldbe possible to use the existing CTE mechanics to do it,>>> but noooo... the existing code is actively and
ferociouslyhostile to>>> the idea of adding new CTEs from within the planner.)
 
>> That's unfortunate.
Tom> I'm less than convinced that it's true ...

Maybe you can figure out how, but I certainly didn't see a reasonable way.

I would also question one aspect of the desirability - using the CTE
mechanism has the downside of needing an extra tuplestore with the
full input data set in it, whereas the chain mechanism only has
aggregated data in its tuplestore which should be much smaller.

-- 
Andrew (irc:RhodiumToad)



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

Предыдущее
От: Petr Jelinek
Дата:
Сообщение: Re: pg_background (and more parallelism infrastructure patches)
Следующее
От: Petr Jelinek
Дата:
Сообщение: Add shutdown_at_recovery_target option to recovery.conf