thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-09-19 16:20:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add has_nullable to fix bug with not using null map in arrayIntersect.cpp

Browse Source
This commit is contained in:
Peter Nguyen 2024-08-31 13:41:21 -06:00
parent 693abc941e
commit 181335f88b
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/Functions/array/arrayIntersect.cpp
View File

@ -514,6 +514,7 @@ ColumnPtr FunctionArrayIntersect<Mode>::execute(const UnpackedArrays & arrays, M
auto rows = arrays.base_rows;
bool all_nullable = true;
bool has_nullable = false;
std::vector<const ColumnType *> columns;
columns.reserve(args);
@ -529,6 +530,8 @@ ColumnPtr FunctionArrayIntersect<Mode>::execute(const UnpackedArrays & arrays, M
if (!arg.null_map)
all_nullable = false;
else
has_nullable = true;
}
auto & result_data = static_cast<ColumnType &>(*result_data_ptr);
@ -608,7 +611,7 @@ ColumnPtr FunctionArrayIntersect<Mode>::execute(const UnpackedArrays & arrays, M
if constexpr (std::is_same_v<Mode, ArrayModeUnion>)
{
use_null_map = has_a_null;
use_null_map = has_nullable;
for (auto & p : map)
{
typename Map::LookupResult pair = map.find(p.getKey());