wsjz commented on code in PR #13867:
URL: https://github.com/apache/doris/pull/13867#discussion_r1021205526


##########
be/src/vec/exec/format/parquet/vparquet_reader.cpp:
##########
@@ -355,25 +438,34 @@ Status ParquetReader::_init_row_group_readers() {
     return Status::OK();
 }
 
-Status ParquetReader::_filter_row_groups() {
-    if (_total_groups == 0 || _t_metadata->num_rows == 0 || _range_size < 0) {
+Status ParquetReader::_filter_row_groups(const bool& enabled,
+                                         std::vector<RowGroupIndex>& 
group_indexes) {
+    if (enabled && (_total_groups == 0 || _t_metadata->num_rows == 0 || 
_range_size < 0)) {
         return Status::EndOfFile("No row group need read");
     }
+    int32_t start_row_id = 0;

Review Comment:
   whether it is enabled or not,we will generate group index for read in 
parallel or other optimizations



-- 
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