Re: An oversight in ExecInitAgg for grouping sets

Поиск
Список
Период
Сортировка
От Richard Guo
Тема Re: An oversight in ExecInitAgg for grouping sets
Дата
Msg-id CAMbWs49nMd1up4BOq=gXutTJintTUpe1u3qxvhRhPj9=9h5U=g@mail.gmail.com
обсуждение исходный текст
Ответ на An oversight in ExecInitAgg for grouping sets  (Richard Guo <guofenglinux@gmail.com>)
Ответы Re: An oversight in ExecInitAgg for grouping sets  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-hackers

On Thu, Dec 22, 2022 at 2:02 PM Richard Guo <guofenglinux@gmail.com> wrote:
I happened to notice $subject.  It happens when we build eqfunctions for
each grouping set.

     /* for each grouping set */
     for (int k = 0; k < phasedata->numsets; k++)
     {
         int         length = phasedata->gset_lengths[k];

         if (phasedata->eqfunctions[length - 1] != NULL)
             continue;

         phasedata->eqfunctions[length - 1] =
             execTuplesMatchPrepare(scanDesc,
                                    length,
                                    aggnode->grpColIdx,
                                    aggnode->grpOperators,
                                    aggnode->grpCollations,
                                    (PlanState *) aggstate);
     }

If it is an empty grouping set, its length will be zero, and accessing
phasedata->eqfunctions[length - 1] is not right.

I think we can just skip building the eqfunctions for empty grouping
set.
 
Attached is a trivial patch for the fix.

Thanks
Richard
Вложения

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

Предыдущее
От: "Hayato Kuroda (Fujitsu)"
Дата:
Сообщение: RE: Time delayed LR (WAS Re: logical replication restrictions)
Следующее
От: Peter Geoghegan
Дата:
Сообщение: Re: New strategies for freezing, advancing relfrozenxid early