From 77d79e3617dbde0033f1c438591480f7b1264a01 Mon Sep 17 00:00:00 2001 From: liaoxin Date: Thu, 28 Mar 2024 07:29:41 +0000 Subject: [PATCH] [fix](merge-iterator) fix NOT_IMPLEMENTED_ERROR when read next block view --- be/src/vec/olap/vgeneric_iterators.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/be/src/vec/olap/vgeneric_iterators.cpp b/be/src/vec/olap/vgeneric_iterators.cpp index 2d45c884a78e41..c973a7131f9ff7 100644 --- a/be/src/vec/olap/vgeneric_iterators.cpp +++ b/be/src/vec/olap/vgeneric_iterators.cpp @@ -422,9 +422,9 @@ Status VUnionIterator::current_block_row_locations(std::vector* loc RowwiseIteratorUPtr new_merge_iterator(std::vector&& 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(std::move(inputs), sequence_id_idx, is_unique, is_reverse, merged_rows); }