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

Disable 'Dictionary' storage due to count() can return incorrect result

Browse Source 
see test_dictionaries_redis/test_long.py::test_redis_dict_long
This commit is contained in:
Igor Nikonov 2023-04-13 20:23:59 +00:00
parent bb60f10035
commit fcd2eaea72
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Storages/IStorage.cpp
View File

@ -138,7 +138,9 @@ void IStorage::read(
const size_t output_ports = pipe.numOutputPorts();
const auto storage_name = getName();
if (output_ports > 0 && output_ports < num_streams && storage_name != "SystemNumbers" && storage_name != "SourceFromSingleChunk"
&& storage_name != "SystemOne")
&& storage_name != "SystemOne"
&& storage_name != "Dictionary") /// dictionaries are disabled due to some bug, count() can return wrong result
/// see test_dictionaries_redis/test_long.py::test_redis_dict_long
pipe.resize(num_streams);
readFromPipe(query_plan, std::move(pipe), column_names, storage_snapshot, query_info, context, getName());