godfreyhe commented on issue #11340: [Flink 14338] Upgrade Calcite version to 1.22 for Flink SQL URL: https://github.com/apache/flink/pull/11340#issuecomment-598618175 > > Thanks @danny0405 so much for this huge work, I left some comments. I notice there are a lot of alias name in the plans are dropped, does calcite not care about alias changes? > > Yes, the alias actually does not affect the semantics of the plan, we can reuse more sub-graph if we ignore that, this is the background CALCITE-3713 fired. I agree with you that the alias does not affect the semantics, but the users will confuse in SQL client if the alias of sink column is dropped.
---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services
