Mailing-List: contact dev-help@phoenix.apache.org; run by ezmlm
Precedence: bulk
Reply-To: dev@phoenix.apache.org
Date: Fri, 28 Apr 2017 06:26:04 +0000 (UTC)
From: "Ankit Singhal (JIRA)" <jira@apache.org>
To: dev@phoenix.apache.org
Message-ID: <JIRA.13066310.1493035674000.75515.1493360764032@Atlassian.JIRA>
In-Reply-To: <JIRA.13066310.1493035674000@Atlassian.JIRA>
References: <JIRA.13066310.1493035674000@Atlassian.JIRA> <JIRA.13066310.1493035674660@jira-lw-us.apache.org>
Subject: [jira] [Commented] (PHOENIX-3809) CURRENT_DATE() (with parentheses)
 is illegal in Calcite
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
archived-at: Fri, 28 Apr 2017 06:26:08 -0000


    [ https://issues.apache.org/jira/browse/PHOENIX-3809?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15988283#comment-15988283 ] 

Ankit Singhal commented on PHOENIX-3809:
----------------------------------------

bq. I think having a version that allows parentheses would ease the transition from 4.x to 5.0, so supporting it would be useful.
Raised pull request for the same. 
https://github.com/apache/calcite/pull/438

though it's not a standard SQL but CURRENT_DATE (with and without parentheses) is also allowed in Mysql, so IMO continuing supporting at our side would not be that bad.


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