viirya commented on code in PR #5260:
URL: https://github.com/apache/arrow-datafusion/pull/5260#discussion_r1104069998


##########
datafusion/core/src/physical_plan/file_format/parquet.rs:
##########
@@ -834,82 +834,116 @@ mod tests {
         parquet_exec: Arc<ParquetExec>,
     }
 
-    /// writes each RecordBatch as an individual parquet file and re-reads
-    /// the data back. Returns the data as [RecordBatch]es
-    async fn round_trip_to_parquet(
-        batches: Vec<RecordBatch>,
+    /// round-trip record batches by writing each individual RecordBatch to
+    /// a parquet file and then reading that parquet file with the specified
+    /// options

Review Comment:
   ```suggestion
       /// round-trip record batches by writing each individual RecordBatch to
       /// a parquet file and then reading that parquet file with the specified
       /// options. If page_index_predicate is set to `true`, all RecordBatches
       /// are written into a parquet file instead.
   ```



-- 
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: github-unsubscr...@arrow.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to