github-actions[bot] commented on code in PR #14900:
URL: https://github.com/apache/doris/pull/14900#discussion_r1042091880
##########
be/src/vec/olap/vertical_merge_iterator.cpp:
##########
@@ -529,10 +571,12 @@ Status VerticalMaskMergeIterator::init(const
StorageReadOptions& opts) {
// interfaces to create vertical merge iterator
std::shared_ptr<RowwiseIterator> new_vertical_heap_merge_iterator(
- const std::vector<RowwiseIterator*>& inputs, size_t ori_return_cols,
KeysType keys_type,
- uint32_t seq_col_idx, RowSourcesBuffer* row_sources) {
- return std::make_shared<VerticalHeapMergeIterator>(std::move(inputs),
ori_return_cols,
- keys_type, seq_col_idx,
row_sources);
+ const std::vector<RowwiseIterator*>& inputs, const std::vector<bool>&
iterator_init_flag,
+ size_t ori_return_cols, KeysType keys_type, uint32_t seq_col_idx,
+ RowSourcesBuffer* row_sources) {
+ return std::make_shared<VerticalHeapMergeIterator>(
+ std::move(inputs), iterator_init_flag, ori_return_cols, keys_type,
Review Comment:
warning: std::move of the const variable 'inputs' has no effect; remove
std::move() or make the variable non-const [performance-move-const-arg]
```suggestion
inputs, iterator_init_flag, ori_return_cols, keys_type,
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscr...@doris.apache.org
For additional commands, e-mail: commits-h...@doris.apache.org
