adriangb commented on PR #18938: URL: https://github.com/apache/datafusion/pull/18938#issuecomment-3628286336
The current status from my perspective is that I pushed back on changing the 2 state logic of pushdown to 3 stats because (1) it generates a lot of API churn and (2) it's not clear that it covers all cases e.g. does bloom filters count as stats pruning or not, etc. An approach that only targets dynamic filters and answers the question "is someone keeping a reference to this" at least resolves the case where no one is keeping a reference to the filter and thus it can't possibly be used, while making the public API changes much smaller. -- 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]
