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


##########
datafusion/core/src/physical_optimizer/pruning.rs:
##########
@@ -251,8 +266,12 @@ fn is_always_true(expr: &Arc<dyn PhysicalExpr>) -> bool {
         .unwrap_or_default()
 }
 
-/// Records for which columns statistics are necessary to evaluate a
-/// pruning predicate.
+/// Describes which columns statistics are necessary to evaluate a
+/// [`PruningPredicate`].
+///
+/// This structure permits reading and creating the minimum number statistics,
+/// which is important since statistics may be non trivial to read (e.g. large
+/// strings or when there are 1000s of columns.

Review Comment:
   ```suggestion
   /// which is important since statistics may be non trivial to read (e.g. 
large
   /// strings or when there are 1000s of columns).
   ```



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to