database/doris
2
0
Fork 0
Code Issues Pull Requests Actions 3 Packages Projects Releases Wiki Activity

[fix][Vectorized] Fix exchange node merge sort null first order wrong (#8291)

Browse Source
This commit is contained in:
HappenLee
2022-03-02 10:19:06 +08:00
committed by GitHub
parent efe88be087
commit b241bc4e9d
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
be/src/vec/core/sort_cursor.h
View File

@ -118,7 +118,7 @@ struct ReceiveQueueSortCursorImpl : public SortCursorImpl {
desc.resize(ordering_expr.size());
for (int i = 0; i < desc.size(); i++) {
desc[i].direction = is_asc_order[i] ? 1 : -1;
desc[i].nulls_direction = nulls_first[i] ? -1 : 1;
desc[i].nulls_direction = nulls_first[i] ? -desc[i].direction : desc[i].direction;
}
_is_eof = !has_next_block();
}

2
be/src/vec/runtime/vsorted_run_merger.cpp
View File

@ -41,7 +41,7 @@ VSortedRunMerger::VSortedRunMerger(const std::vector<VExprContext *>& ordering_e
}
Status VSortedRunMerger::prepare(const vector<BlockSupplier>& input_runs, bool parallel) {
for (const auto &supplier : input_runs) {
for (const auto& supplier : input_runs) {
_cursors.emplace_back(supplier, _ordering_expr, _is_asc_order, _nulls_first);
}