nielspardon commented on PR #4230: URL: https://github.com/apache/calcite/pull/4230#issuecomment-2707104844
> Interesting that this does not affect any existing tests. Indeed, I was also a bit suprised that this code has been there for 9 years and nobody has encountered this before. One thing I found was that when you start with a SQL statement through the SQL parser it combines the column reduction and renaming into a single `LogicalProject` which ends up within the `RelRoot.rel` so through the normal test approach in e.g. `SqlToRelConverterTest` you will not encounter this scenario. -- 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]
