alamb commented on issue #16188: URL: https://github.com/apache/datafusion/issues/16188#issuecomment-2992278637
> Is there a duplication with push_down_filter in the logical optimizer? In my opinion there is duplication in the sense that they both do similar things (push filters down in the plan) but for the reasons that @adriangb mentions we can't combine them It is interesting that InfluxData also ended up with its own physical rule to push filters down (in our case for other reasons) Thus I think it is inevitable that we'll have similar functonality in both the Logical and Physical plan layers -- 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
