| От | 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)
|
| Список | 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 по дате отправления:
Сайт использует файлы cookie для корректной работы и повышения удобства. Нажимая кнопку «Принять» или продолжая пользоваться сайтом, вы соглашаетесь на их использование в соответствии с Политикой в отношении обработки cookie ООО «ППГ», в том числе на передачу данных из файлов cookie сторонним статистическим и рекламным службам. Вы можете управлять настройками cookie через параметры вашего браузера