adriangb commented on PR #16424:
URL: https://github.com/apache/datafusion/pull/16424#issuecomment-3006625465

   @xudong963 @alamb I've re-organized this to incorporate 
https://github.com/apache/datafusion/pull/16549.
   
   Sadly I did not catch in that PR that we were putting everything in `lib.rs` 
which I felt now is too bloated if I put `FilePruner` in there. So I moved 
`PruningPredicate` & co to `pruning_predicate.rs` - hence the huge diff line 
count.
   
   I'll also point out that this now only does the extra work if it has either 
a dynamic filter OR the file has statistics already collected.


-- 
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...@datafusion.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org
For additional commands, e-mail: github-h...@datafusion.apache.org

Reply via email to