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

Merge pull request #19788 from ClickHouse/array-enumerate-extended-bug

Browse Source 
Fix trivial bug in arrayEnumerateUniq
This commit is contained in:
alexey-milovidov 2021-01-29 14:58:38 +03:00 committed by GitHub
commit 0679d81c0d
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 18 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/Functions/array/arrayEnumerateExtended.h
View File

@ -353,6 +353,9 @@ bool FunctionArrayEnumerateExtended<Derived>::execute128bit(
keys_bytes += key_sizes[j];
}
if (keys_bytes > 16)
return false;
executeMethod<MethodFixed>(offsets, columns, key_sizes, nullptr, res_values);
return true;
}

2
tests/queries/0_stateless/01677_array_enumerate_bug.reference Normal file
View File

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

13
tests/queries/0_stateless/01677_array_enumerate_bug.sql Normal file
View File

@ -0,0 +1,13 @@
-- there was a bug - missing check of the total size of keys for the case with hash table with 128bit key.
SELECT arrayEnumerateUniq(arrayEnumerateUniq([toInt256(10), toInt256(100), toInt256(2)]), [toInt256(123), toInt256(1023), toInt256(123)]);
SELECT arrayEnumerateUniq(
[111111, 222222, 333333],
[444444, 555555, 666666],
[111111, 222222, 333333],
[444444, 555555, 666666],
[111111, 222222, 333333],
[444444, 555555, 666666],
[111111, 222222, 333333],
[444444, 555555, 666666]);