thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-23 08:02:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix comments

Browse Source
This commit is contained in:
Dmitry Novik 2024-01-12 09:42:41 +01:00 committed by GitHub
parent 322d9478b0
commit 91f4b20e02
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/Interpreters/InterpreterSelectQuery.cpp
View File

@ -2985,7 +2985,7 @@ void InterpreterSelectQuery::executeWindow(QueryPlan & query_plan)
// has suitable sorting. Also don't create sort steps when there are no // has suitable sorting. Also don't create sort steps when there are no
// columns to sort by, because the sort nodes are confused by this. It // columns to sort by, because the sort nodes are confused by this. It
// happens in case of `over ()`. // happens in case of `over ()`.
// Even if full_sort_description of both windows match, we in case of different // Even if full_sort_description of both windows match, in case of different
// partitioning we need to add a SortingStep to reshuffle data in the streams. // partitioning we need to add a SortingStep to reshuffle data in the streams.
bool need_sort = !window.full_sort_description.empty(); bool need_sort = !window.full_sort_description.empty();
if (need_sort && i != 0) if (need_sort && i != 0)

2
src/Planner/Planner.cpp
View File

@ -908,7 +908,7 @@ void addWindowSteps(QueryPlan & query_plan,
* has suitable sorting. Also don't create sort steps when there are no * has suitable sorting. Also don't create sort steps when there are no
* columns to sort by, because the sort nodes are confused by this. It * columns to sort by, because the sort nodes are confused by this. It
* happens in case of `over ()`. * happens in case of `over ()`.
* Even if full_sort_description of both windows match, we in case of different * Even if full_sort_description of both windows match, in case of different
* partitioning we need to add a SortingStep to reshuffle data in the streams. * partitioning we need to add a SortingStep to reshuffle data in the streams.
*/ */