Skip to content

Commit

Permalink
[fix](merge-iterator) fix NOT_IMPLEMENTED_ERROR when read next block …
Browse files Browse the repository at this point in the history
…view (#32961) (#33146)
  • Loading branch information
liaoxin01 authored Apr 2, 2024
1 parent 9ac0ddd commit f52183d
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions be/src/vec/olap/vgeneric_iterators.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -420,9 +420,9 @@ Status VUnionIterator::current_block_row_locations(std::vector<RowLocation>* loc
RowwiseIteratorUPtr new_merge_iterator(std::vector<RowwiseIteratorUPtr>&& inputs,
int sequence_id_idx, bool is_unique, bool is_reverse,
uint64_t* merged_rows) {
if (inputs.size() == 1) {
return std::move(inputs[0]);
}
// when the size of inputs is 1, we also need to use VMergeIterator, because the
// next_block_view function only be implemented in VMergeIterator. The reason why
// the size of inputs is 1 is that the segment was filtered out by zone map or others.
return std::make_unique<VMergeIterator>(std::move(inputs), sequence_id_idx, is_unique,
is_reverse, merged_rows);
}
Expand Down

0 comments on commit f52183d

Please sign in to comment.