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

Fix weakHash

Browse Source
This commit is contained in:
Nikolai Kochetov 2020-03-20 20:05:04 +03:00
parent ebba6a6100
commit 75192d44aa
1 changed files with 11 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
dbms/src/Common/HashTable/Hash.h
View File

@ -85,7 +85,7 @@ inline UInt32 updateWeakHash32(const DB::UInt8 * pos, size_t size, DB::UInt32 u
}
const auto * end = pos + size;
while (pos + 8 < end)
while (pos + 8 <= end)
{
auto word = unalignedLoad<UInt64>(pos);
updated_value = intHashCRC32(word, updated_value);
@ -93,9 +93,16 @@ inline UInt32 updateWeakHash32(const DB::UInt8 * pos, size_t size, DB::UInt32 u
pos += 8;
}
auto word = unalignedLoad<UInt64>(end - 8);
word &= (~UInt64(0)) << DB::UInt8(8 * (8 - (end - pos)));
return intHashCRC32(word, updated_value);
if (pos < end)
{
DB::UInt8 tail_size = end - pos;
auto word = unalignedLoad<UInt64>(end - 8);
word &= (~UInt64(0)) << DB::UInt8(8 * (8 - tail_size));
word |= tail_size;
updated_value = intHashCRC32(word, updated_value);
}
return updated_value;
}
template <typename T>