2010YOUY01 commented on PR #19609: URL: https://github.com/apache/datafusion/pull/19609#issuecomment-3727237478
> If we do want to proceed, I think the first thing we should do is figure out how we will eventually unify the existing APIs that have overlap (specifically `PruningPredicate` and `PhysicalExpr::evalute_bounds`, possibly the propagate statistics code too) Regarding `PruningPredicate`, I think its major API can eventually remain the same, while its implementation should be replaced with this statistics-propagation-based pruning. Some minor API changes are inevitable. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
