alamb commented on PR #4465: URL: https://github.com/apache/arrow-datafusion/pull/4465#issuecomment-1441790923
> Yes, this is one of the purposes. As you said above, we can make PR https://github.com/apache/arrow-datafusion/pull/5366 to eliminate the redundant projections that appear in this PR. Awesome -- I think as long a the redundant projections are removed by the final plan it is fine to introduce them in an earlier pass 👍 > Another purpose is to split one part (eliminate projection) of this PR into one new rule. This make rule keep it as simple as possible, don't do too many things at once in one rule. And this PR will be more simple than 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]
