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 #39812 from guowangy/applyFunction-multi-thread

Browse Source 
KeyCondition: optimize applyFunction in multi-thread scenario
This commit is contained in:
Nikolay Degterinsky 2022-08-04 01:36:47 +02:00 committed by GitHub
commit 71cb055ecc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Storages/MergeTree/KeyCondition.cpp
View File

@ -607,9 +607,9 @@ static FieldRef applyFunction(const FunctionBasePtr & func, const DataTypePtr &
result_idx = i;
}
ColumnsWithTypeAndName args{(*columns)[field.column_idx]};
if (result_idx == columns->size())
{
ColumnsWithTypeAndName args{(*columns)[field.column_idx]};
field.columns->emplace_back(ColumnWithTypeAndName {nullptr, func->getResultType(), result_name});
(*columns)[result_idx].column = func->execute(args, (*columns)[result_idx].type, columns->front().column->size());
}