mirror of
https://github.com/ClickHouse/ClickHouse.git
synced 2024-09-21 01:00:48 +00:00
Fixed code review issues
This commit is contained in:
parent
1fea19846b
commit
42201d3e30
@ -302,8 +302,6 @@ Aggregator::Aggregator(const Params & params_)
|
|||||||
|
|
||||||
void Aggregator::compileAggregateFunctionsIfNeeded()
|
void Aggregator::compileAggregateFunctionsIfNeeded()
|
||||||
{
|
{
|
||||||
std::cerr << "Aggregator::compileAggregateFunctions" << std::endl;
|
|
||||||
|
|
||||||
static std::unordered_map<UInt128, UInt64, UInt128Hash> aggregate_functions_description_to_count;
|
static std::unordered_map<UInt128, UInt64, UInt128Hash> aggregate_functions_description_to_count;
|
||||||
static std::mutex mtx;
|
static std::mutex mtx;
|
||||||
|
|
||||||
@ -346,13 +344,6 @@ void Aggregator::compileAggregateFunctionsIfNeeded()
|
|||||||
if (functions_to_compile.empty())
|
if (functions_to_compile.empty())
|
||||||
return;
|
return;
|
||||||
|
|
||||||
std::cerr << "Functions to compile " << functions_to_compile.size() << std::endl;
|
|
||||||
|
|
||||||
for (auto & function : functions_to_compile)
|
|
||||||
{
|
|
||||||
std::cerr << "Function " << function.function->getDescription() << " offset " << function.aggregate_data_offset << std::endl;
|
|
||||||
}
|
|
||||||
|
|
||||||
SipHash aggregate_functions_description_hash;
|
SipHash aggregate_functions_description_hash;
|
||||||
aggregate_functions_description_hash.update(functions_description);
|
aggregate_functions_description_hash.update(functions_description);
|
||||||
|
|
||||||
|
Loading…
Reference in New Issue
Block a user