suibianwanwank commented on PR #4177: URL: https://github.com/apache/calcite/pull/4177#issuecomment-2649688630
I've already Approved it. Also currently we don't support SOME, ALL, ANY in RelToSqlConverter, if it is supported here in the future, it may require more condition. I think the cause here would be that the alias in a related query needs to take the alias of the top query. It would be great if there was a way to handle this consistently. But as far as test results go, it's fine 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]
