asolimando commented on code in PR #4197:
URL: https://github.com/apache/calcite/pull/4197#discussion_r1957081914


##########
core/src/main/java/org/apache/calcite/sql/SqlUnnestOperator.java:
##########
@@ -121,14 +121,7 @@ private static boolean 
allowAliasUnnestItems(SqlOperatorBinding operatorBinding)
 
   @Override public void unparse(SqlWriter writer, SqlCall call, int leftPrec,
       int rightPrec) {
-    if (call.operandCount() == 1

Review Comment:
   Thanks for researching and fixing this @NobiGo, +1 if Calcite is the only 
exception, unless there is a good reason to, we generally try to comply with 
all major DBs.
   
   May I only suggest to put this information in the Jira ticket description 
(the list of databases you tried), it might come handy later and there is less 
chance people will dig this information up from the PR.



-- 
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]

Reply via email to