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

Drop check for getPositionByName() (it already has it) in ExpressionActions

Browse Source
This commit is contained in:
Azat Khuzhin 2020-05-02 14:45:01 +03:00
parent 88d08f9f39
commit 1f45c4302a
1 changed files with 0 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/Interpreters/ExpressionActions.cpp
View File

@ -342,11 +342,7 @@ void ExpressionAction::execute(Block & block, bool dry_run, ExtraBlockPtr & not_
{ {
ColumnNumbers arguments(argument_names.size()); ColumnNumbers arguments(argument_names.size());
for (size_t i = 0; i < argument_names.size(); ++i) for (size_t i = 0; i < argument_names.size(); ++i)
{
if (!block.has(argument_names[i]))
throw Exception("Not found column: '" + argument_names[i] + "'", ErrorCodes::NOT_FOUND_COLUMN_IN_BLOCK);
arguments[i] = block.getPositionByName(argument_names[i]); arguments[i] = block.getPositionByName(argument_names[i]);
}
size_t num_columns_without_result = block.columns(); size_t num_columns_without_result = block.columns();
block.insert({ nullptr, result_type, result_name}); block.insert({ nullptr, result_type, result_name});