[jira] [Commented] (CALCITE-1829) Add TIME/TIMESTAMP/DATE datatype handling to RexImplicationChecker

["Julian Hyde (JIRA)" <ji...@apache.org>]

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

Julian Hyde commented on CALCITE-1829:
--------------------------------------

Am I correct that this feature would affect what SQL expressions are valid? If so, I think you should explain it in SQL terms.

> Add TIME/TIMESTAMP/DATE datatype handling to RexImplicationChecker
> ------------------------------------------------------------------
>
>                 Key: CALCITE-1829
>                 URL: https://issues.apache.org/jira/browse/CALCITE-1829
>             Project: Calcite
>          Issue Type: Bug
>            Reporter: MinJi Kim
>            Assignee: Julian Hyde
>
> In RexImplicationChecker, we support a lot of operators and types, but it looks like time/timestamp/date datatypes are not supported.  In RexImplicationCheckerTest, there are tests for each of the aforementioned datatypes, but are disabled.
> {code}
> RexImplicationCheckerTest.testSimpleDate
> RexImplicationCheckerTest.testSimpleTime
> RexImplicationCheckerTest.testSimpleTimestamp
> {code}
> Running these ignored tests, I get the following error messages.
> {code}
> 2017-06-02 17:13:10,028 [main] WARN  - Exception thrown while checking if => >=($8, 2017-06-03): java.sql.Date cannot be cast to java.lang.Integer
> 2017-06-02 17:13:10,075 [main] WARN  - Exception thrown while checking if => <=($9, 00:13:10): java.util.GregorianCalendar cannot be cast to java.lang.Long
> 2017-06-02 17:13:10,100 [main] WARN  - Exception thrown while checking if => <=($9, 2017-06-03 00:13:10): java.util.GregorianCalendar cannot be cast to java.lang.Long
> {code}



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)