thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-09-20 00:30:49 +00:00
Code Issues Packages Projects Releases Wiki Activity

dbms: development [#CONV-2944].

Browse Source
This commit is contained in:
Alexey Milovidov 2011-09-25 06:42:59 +00:00
parent 3934964e44
commit 6cccbca00a
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
dbms/src/Interpreters/InterpreterSelectQuery.cpp
View File

@ -93,14 +93,14 @@ BlockInputStreamPtr InterpreterSelectQuery::execute()
limit_offset = boost::get<UInt64>(dynamic_cast<ASTLiteral &>(*query.limit_offset).value);
}
bool has_aggregates = expression->hasAggregates();
bool need_aggregate = expression->hasAggregates() || query.group_expression_list;
/** Оптимизация - если не указаны WHERE, GROUP, HAVING, ORDER, но указан LIMIT, и limit + offset < max_block_size,
* то в качестве размера блока будем использовать limit + offset (чтобы не читать из таблицы больше, чем запрошено).
*/
size_t block_size = max_block_size;
if (!query.where_expression && !query.group_expression_list && !query.having_expression && !query.order_expression_list
&& query.limit_length && !has_aggregates && limit_length + limit_offset < block_size)
&& query.limit_length && !need_aggregate && limit_length + limit_offset < block_size)
{
block_size = limit_length + limit_offset;
}
@ -123,7 +123,7 @@ BlockInputStreamPtr InterpreterSelectQuery::execute()
}
/// Если есть GROUP BY - сначала выполним часть выражения, необходимую для его вычисления
if (has_aggregates)
if (need_aggregate)
{
expression->markBeforeAndAfterAggregation(PART_BEFORE_AGGREGATING, PART_AFTER_AGGREGATING);