[
https://issues.apache.org/jira/browse/PHOENIX-3809?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15982190#comment-15982190
]
James Taylor commented on PHOENIX-3809:
---------------------------------------
I think having a version that allows parentheses would ease the transition from
4.x to 5.0, so supporting it would be useful. Only if it'd be a lot of work to
support, would I be more inclined to change all the tests and disallow usage
with parentheses. I think it'd be one of those things we'd discourage from
being used, though.
> CURRENT_DATE() (with parentheses) is illegal in Calcite
> -------------------------------------------------------
>
> Key: PHOENIX-3809
> URL: https://issues.apache.org/jira/browse/PHOENIX-3809
> Project: Phoenix
> Issue Type: Bug
> Reporter: Ankit Singhal
> Assignee: Ankit Singhal
> Labels: calcite
>
> Calcite doesn't allow system functions to be specified with parenthesis who
> doesn't accept arguments.
> For eg:-
> CURRENT_DATE() is illegal whereas CURRENT_DATE is expected.
> At validation level:- SqlValidatorImpl#validateCall
> {code}
> if ((call.operandCount() == 0)
> && (operator.getSyntax() == SqlSyntax.FUNCTION_ID)
> && !call.isExpanded()) {
> // For example, "LOCALTIME()" is illegal. (It should be
> // "LOCALTIME", which would have been handled as a
> // SqlIdentifier.)
> throw handleUnresolvedFunction(call, (SqlFunction) operator,
> ImmutableList.<RelDataType>of(), null);
> }
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
