BiteTheDDDDt commented on a change in pull request #8299:
URL: https://github.com/apache/incubator-doris/pull/8299#discussion_r818431531
##########
File path: fe/fe-core/src/main/cup/sql_parser.cup
##########
@@ -4743,6 +4743,10 @@ function_call_expr ::=
RESULT = new FunctionCallExpr(fn_name, params);
}
:}
+ | function_name:fn_name LPAREN function_params:left_params RPAREN LPAREN
function_params:params RPAREN
Review comment:
1. From the point of view of function capabilities, single parentheses
can meet the needs.
For example, ```intersect count``` is currently implemented in this way.
2. From the perspective of grammar compatibility, it depends on whether you
want to follow the consistent syntax style of Doris or cater to CK.
In fact, much of the grammar of CK is not generic.
--
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
