thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-27 01:51:59 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #46569 from ClickHouse/add-bug

Browse Source 
Fix processing Const(LowCardinality) in arrayMap
This commit is contained in:
Kruglov Pavel 2023-02-21 13:11:06 +01:00 committed by GitHub
commit 36870f0801
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/Functions/array/FunctionArrayMapped.h
View File

@ -7,6 +7,7 @@
#include <Columns/ColumnFunction.h>
#include <Columns/ColumnMap.h>
#include <Columns/ColumnNullable.h>
#include <Columns/ColumnLowCardinality.h>
#include <Columns/IColumn.h>
#include <Common/Exception.h>
@ -393,9 +394,15 @@ public:
replicated_column_function->appendArguments(arrays);
auto lambda_result = replicated_column_function->reduce();
/// Convert LowCardinality(T) -> T and Const(LowCardinality(T)) -> Const(T),
/// because we removed LowCardinality from return type of lambda expression.
if (lambda_result.column->lowCardinality())
lambda_result.column = lambda_result.column->convertToFullColumnIfLowCardinality();
if (const auto * const_column = checkAndGetColumnConst<ColumnLowCardinality>(lambda_result.column.get()))
lambda_result.column = const_column->removeLowCardinality();
if (Impl::needBoolean())
{
/// If result column is Nothing or Nullable(Nothing), just create const UInt8 column with 0 value.

1
tests/queries/0_stateless/02667_array_map_const_low_cardinality.reference Normal file
View File

@ -0,0 +1 @@
[2]

1
tests/queries/0_stateless/02667_array_map_const_low_cardinality.sql Normal file
View File

@ -0,0 +1 @@
SELECT arrayMap(x -> (toLowCardinality(1) + 1::Nullable(UInt8)), [1]);