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

Bug fix for funciton fuzzBits

Browse Source
This commit is contained in:
hexiaoting 2020-11-16 11:46:05 +08:00
parent 8d3858fc22
commit 26a529fc32
3 changed files with 40 additions and 10 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
src/Functions/fuzzBits.cpp
View File

@ -99,6 +99,10 @@ public:
ColumnString::Chars & chars_to = col_to->getChars();
ColumnString::Offsets & offsets_to = col_to->getOffsets();
size_t col_in_rows = col_in->getOffsets().size();
if (col_in_rows >= input_rows_count)
{
chars_to.resize(col_in->getChars().size());
// TODO: Maybe we can share `col_in->getOffsets()` to `offsets_to.resize` like clever pointers? They are same
offsets_to.resize(input_rows_count);
@ -112,6 +116,24 @@ public:
offsets_to[i] = col_in->getOffsets()[i];
ptr_to[offsets_to[i] - 1] = 0;
}
}
else
{
assert(col_in_rows == 1);
chars_to.resize(col_in->getChars().size() * input_rows_count);
offsets_to.resize(input_rows_count);
size_t offset = col_in->getOffsets()[0];
const auto * ptr_in = col_in->getChars().data();
auto * ptr_to = chars_to.data();
for (size_t i = 0; i < input_rows_count; ++i)
{
fuzzBits(ptr_in, ptr_to + i * offset, offset, inverse_probability);
offsets_to[i] = (i + 1) * offset;
ptr_to[offsets_to[i] - 1] = 0;
}
}
return col_to;
}

5
tests/queries/0_stateless/01585_fuzz_bits_with_bugfix.reference Normal file
View File

@ -0,0 +1,5 @@
String
String
String
String
String

3
tests/queries/0_stateless/01585_fuzz_bits_with_bugfix.sql Normal file
View File

@ -0,0 +1,3 @@
SELECT toTypeName(fuzzBits('stringstring', 0.5)) from numbers(3);
SELECT toTypeName(fuzzBits('stringstring', 0.5)) from ( SELECT 1 AS x UNION ALL SELECT NULL ) group by x