thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-25 00:52:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #19016 from ClickHouse/fix-ignore-with-lc-arg

Browse Source 
Fix `ignore` function with LowCardinality argument
This commit is contained in:
Nikolai Kochetov 2021-01-14 16:50:54 +03:00 committed by GitHub
commit d8b9278193
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 11 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Functions/ignore.cpp
View File

@ -30,6 +30,10 @@ public:
bool useDefaultImplementationForNulls() const override { return false; }
/// We should never return LowCardinality result, cause we declare that result is always constant zero.
/// (in getResultIfAlwaysReturnsConstantAndHasArguments)
bool useDefaultImplementationForLowCardinalityColumns() const override { return false; }
String getName() const override
{
return name;

1
tests/queries/0_stateless/01652_ignore_and_low_cardinality.reference Normal file
View File

@ -0,0 +1 @@
0

6
tests/queries/0_stateless/01652_ignore_and_low_cardinality.sql Normal file
View File

@ -0,0 +1,6 @@
set allow_suspicious_low_cardinality_types = 1;
CREATE TABLE lc_null_int8_defnull (val LowCardinality(Nullable(Int8)) DEFAULT NULL) ENGINE = MergeTree order by tuple();
SELECT ignore(10, ignore(*), ignore(ignore(-2, 1025, *)), NULL, *), * FROM lc_null_int8_defnull AS values;
SELECT ignore(toLowCardinality(1), toLowCardinality(2), 3);