adriangb commented on issue #17527: URL: https://github.com/apache/datafusion/issues/17527#issuecomment-3283197784
> I think this is intentionally not handled in `handle_child_pushdown_result` 🤔 > > ``` > // Note that we don't check PushdDownPredicate::discrimnant because even if nothing said > // "yes, I can fully evaluate this filter" things might still use it for statistics -> it's worth updating > ``` > cc @adriangb > Yep that's the case. We'd either have to expand the pushdown result enum to three cases: - Exact - Inexact - Unsupported Or build in some notion of "subscribers" to dynamic filters where the thing that's going to use them has to "subscribe". -- 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
