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

Merge pull request #4367 from yandex/unified-sets-fix-lc-cache

Browse Source 
Fix LowCardinality cache for ColumnsHashing
This commit is contained in:
alexey-milovidov 2019-02-12 21:24:52 +03:00 committed by GitHub
commit d8aa74bc64
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 9 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
dbms/src/Common/ColumnsHashing.h
View File

@ -364,7 +364,10 @@ struct HashMethodSingleLowCardinalityColumn : public SingleColumnMethod
} }
if constexpr (has_mapped) if constexpr (has_mapped)
{
mapped_cache[row] = it->second;
return EmplaceResult(it->second, mapped_cache[row], inserted); return EmplaceResult(it->second, mapped_cache[row], inserted);
}
else else
return EmplaceResult(inserted); return EmplaceResult(inserted);
} }

1
dbms/tests/queries/0_stateless/906_low_cardinality_cache.reference Normal file
View File

@ -0,0 +1 @@
100000000 0123456789

5
dbms/tests/queries/0_stateless/906_low_cardinality_cache.sql Normal file
View File

@ -0,0 +1,5 @@
drop table if exists test.lc;
create table test.lc (b LowCardinality(String)) engine=MergeTree order by b;
insert into test.lc select '0123456789' from numbers(100000000);
select count(), b from test.lc group by b;
drop table if exists test.lc;