scrozon opened a new pull request #1998: URL: https://github.com/apache/calcite/pull/1998
adds a project on top of the aggregate when the field names of the new aggregate don't match the field names of the original one The side effects are mostly an extra project node in some of the existing unit tests, which I believe is correct since the existing tests were not respecting the aliases. The only unit test where my changes impacts the structure of the plan is `testPullAggregateThroughUnionWithAlias` where the original version removed the aggregate node in the 2nd input of the union, but I'm not sure why that was the case, both inputs should be aggregated before the union I think. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected]
