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

Reply via email to