yiguolei commented on code in PR #13865:
URL: https://github.com/apache/doris/pull/13865#discussion_r1017368150
##########
be/src/olap/rowset/segment_v2/segment.cpp:
##########
@@ -259,9 +263,8 @@ Status Segment::new_column_iterator(const TabletColumn&
tablet_column, ColumnIte
tablet_column.has_default_value(),
tablet_column.default_value(),
tablet_column.is_nullable(), std::move(type_info),
tablet_column.length(),
tablet_column.precision(), tablet_column.frac()));
- ColumnIteratorOptions iter_opts;
- RETURN_IF_ERROR(default_value_iter->init(iter_opts));
+ RETURN_IF_ERROR(default_value_iter->init(opts));
*iter = default_value_iter.release();
return Status::OK();
}
Review Comment:
not only default iterator should call init here, also other column iterators
on line 271 should call init here.
--
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
