cisaacson commented on issue #10628: URL: https://github.com/apache/datafusion/issues/10628#issuecomment-2129731243
This looks very good, pretty much what I implemented. The only question I have remaining is: If `TableScan` has `filters` why would that not catch all filters? What does `LogicalPlan::Filter` catch that the `TableScan.filters` would not? For my specific use case I only need filters on the tables that are present in the query. -- 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
