thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-12-13 09:52:38 +00:00
Code Issues Packages Projects Releases Wiki Activity

If aggregation step is in order, - do not remove underneath sorting step

Browse Source
This commit is contained in:
Igor Nikonov 2022-12-21 12:55:42 +00:00
parent fdb277245c
commit 919fa9b84e
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/Processors/QueryPlan/Optimizations/removeRedundantOrderBy.cpp
View File

@ -219,6 +219,9 @@ private:
/// (1) aggregation /// (1) aggregation
if (const AggregatingStep * parent_aggr = typeid_cast<AggregatingStep *>(step_affect_order); parent_aggr) if (const AggregatingStep * parent_aggr = typeid_cast<AggregatingStep *>(step_affect_order); parent_aggr)
{ {
if (parent_aggr->inOrder())
return false;
auto const & aggregates = parent_aggr->getParams().aggregates; auto const & aggregates = parent_aggr->getParams().aggregates;
for (const auto & aggregate : aggregates) for (const auto & aggregate : aggregates)
{ {