alamb commented on PR #15301: URL: https://github.com/apache/datafusion/pull/15301#issuecomment-2761509767
> I would still keep the methods on ExecutionPlan to do the pushdown instead of the optimizer rule unless I'm wrong about optimizer rules not being able to deal with LiquidCacheClientExec and such. I don't fully understand the LiquidCacheClient and when it runs / splits the plans into local / remote parts. In general I think we would need some way to handle distributed execution when a dynamic filter requires some state on some other node (like the topK is running on one node but the scan is running on another) -- 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
