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

Fix tupleElement with Null arguments

Browse Source
This commit is contained in:
flynn 2023-02-02 08:48:13 +00:00
parent 5f4726eb2a
commit 8513e5a721
3 changed files with 24 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/Functions/tupleElement.cpp
View File

@ -56,10 +56,9 @@ public:
return true; return true;
} }
ColumnNumbers getArgumentsThatAreAlwaysConstant() const override ColumnNumbers getArgumentsThatAreAlwaysConstant() const override { return {1}; }
{
return {1}; bool useDefaultImplementationForNulls() const override { return false; }
}
bool isSuitableForShortCircuitArgumentsExecution(const DataTypesWithConstInfo & /*arguments*/) const override { return false; } bool isSuitableForShortCircuitArgumentsExecution(const DataTypesWithConstInfo & /*arguments*/) const override { return false; }

2
tests/queries/0_stateless/02541_tuple_element_with_null.reference Normal file
View File

@ -0,0 +1,2 @@
1 1 2 2
\N \N 3 3

19
tests/queries/0_stateless/02541_tuple_element_with_null.sql Normal file
View File

@ -0,0 +1,19 @@
DROP TABLE IF EXISTS test_tuple_element;
CREATE TABLE test_tuple_element
(
tuple Tuple(k1 Nullable(UInt64), k2 UInt64)
)
ENGINE = MergeTree
ORDER BY tuple()
SETTINGS index_granularity = 8192;
INSERT INTO test_tuple_element VALUES (tuple(1,2)), (tuple(NULL, 3));
SELECT
tupleElement(tuple, 'k1', 0) fine_k1_with_0,
tupleElement(tuple, 'k1', NULL) k1_with_null,
tupleElement(tuple, 'k2', 0) k2_with_0,
tupleElement(tuple, 'k2', NULL) k2_with_null
FROM test_tuple_element;
DROP TABLE test_tuple_element;