[ 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)