adriangb commented on PR #20117: URL: https://github.com/apache/datafusion/pull/20117#issuecomment-3843653807
> Did you consider incorporating this logic directly into the `OptimizeProjections`? It seems like this transformation is really just a mechanism to enable `OptimizeProjections` 🤔 `OptimizeProjections` merges projections but it does not push them down. There is no logical optimizer rule for pushing down projections. We could move that part of this rule (the pushdown) into `OptimizeProjections` or a new rule but I’d find it easier to work it all in one place for now. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
