eldenmoon commented on code in PR #49385: URL: https://github.com/apache/doris/pull/49385#discussion_r2009522727
########## be/src/olap/rowset/segment_v2/hierarchical_data_reader.h: ########## @@ -193,10 +193,22 @@ class BaseSparseColumnProcessor : public ColumnIterator { return _sparse_column_reader->init(opts); } + // When performing compaction, multiple columns are extracted from the sparse columns, + // and the sparse columns only need to be read once. + // So we need to cache the sparse column and reuse it. + // The cache is only used when the compaction reader is used. + bool has_sparse_column_cache() const { Review Comment: need ut in further test -- 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