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:
Single bracket cant support `Parametric Aggregate Functions` grammar
from ck.
Single bracket cannot represent 2 variable length arguments. But maybe we
can use some grammar like `(n,a1,a2...an,m,b1,b2..bm)` to represent it.
--
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]
