eejbyfeldt commented on PR #12281: URL: https://github.com/apache/datafusion/pull/12281#issuecomment-2334003208
> It seems like the `with_projection()` API could end up being a method of the `ExecutionPlan` in the long run, wdyt? (It would improve the performance of all operators which internally refer to some columns but its downstream operators do not actually require that column) Probably worth considering in the future. The argument against making it part of ExecutionPlan is that there are some operators like `CoalescePartitionsExec` where it will not make sense. Addressed the comments. -- 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
