Re: [PATCH] Fix division by zero (explain.c)

Поиск
Список
Период
Сортировка
От Tom Lane
Тема Re: [PATCH] Fix division by zero (explain.c)
Дата
Msg-id 16267.1588999511@sss.pgh.pa.us
обсуждение исходный текст
Ответ на Re: [PATCH] Fix division by zero (explain.c)  (James Coleman <jtc331@gmail.com>)
Ответы Re: [PATCH] Fix division by zero (explain.c)  (Ranier Vilela <ranier.vf@gmail.com>)
Список pgsql-hackers
James Coleman <jtc331@gmail.com> writes:
> There are always full sort groups before any prefix groups can happen,
> so we know (even though the tooling doesn't) that the 2nd test can
> never contradict the first.

So maybe an assertion enforcing that would be appropriate?
Untested, but:

-            if (fullsortGroupInfo->groupCount == 0 &&
-                prefixsortGroupInfo->groupCount == 0)
+            if (fullsortGroupInfo->groupCount == 0)
+            {
+                Assert(prefixsortGroupInfo->groupCount == 0);
                 continue;
+            }

            regards, tom lane



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

Предыдущее
От: Peter Geoghegan
Дата:
Сообщение: Re: Should smgrdounlink() be removed?
Следующее
От: Fujii Masao
Дата:
Сообщение: Re: Back-branch minor release notes are up for review