findepi commented on PR #13267: URL: https://github.com/apache/datafusion/pull/13267#issuecomment-2480692128
> Meanwhile, the physical planner can still generate an efficient physical plan with the necessary optimizations (e.g., Join with pushdown). If we can enable this configuration by default without altering any original physical plan, that would be the best-case scenario. That would impose a limit on future optimizations on logical plans, which isn't desirable. > 1. Column references require additional processing to project correctly Can unparsing do this additional processing as needed for unparsing? Generally, is unparsing expected to produce a query plan that is very similar to original query parsed by DF? I hope that's a non goal, but if it is, it should really disable logical plan optimizations and maybe just do nothing more than projection pushdown. -- 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
