Fix fuzzer found error (with totals)#16613
Fix fuzzer found error (with totals)#16613nikitamikhaylov merged 5 commits intoClickHouse:masterfrom
Conversation
robot-clickhouse
added
the
pr-bugfix
|
One test has to be updated. |
|
arrayJoin of Array(Nothing) can be allowed (and was allowed) as soon as no value of type Nothing appears in the result. The issue is when joining with empty array appears during the calculation of WITH TOTALS. But in this case there is nothing special about Array(Nothing) comparing to Arrays of other types. Please check the following query: Maybe you did not really fix the issue and the "kludge" is irrelevant. |
|
Also look at the #14500 It appears that it's just @KochetovNicolai has not completed the fix :) |
|
@KochetovNicolai You forgot to test the case #14500 (review) Totals should simply disappear when joining with empty arrays. |
nikitamikhaylov
force-pushed
the
with_totals_having_assert
branch
from
98e5538 to
aceb82a
Compare
|
Stress test (undefined) broken in master. |
|
@nikitamikhaylov Bad changelog. |
3 participants
I hereby agree to the terms of the CLA available at: https://yandex.ru/legal/cla/?lang=en
Changelog category (leave one):
Changelog entry (a user-readable short description of the changes that goes to CHANGELOG.md):
Fixed #16081.
Detailed description / Documentation draft:
Forbid
arrayJoin's in HAVING clause.