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

Julian Hyde commented on CALCITE-2012:
--------------------------------------

[~jcamachorodriguez], Is this PR ready for review? Do you think this will be 
ready for code freeze on Nov 27?

> Replace LocalInterval by Interval in Druid adapter
> --------------------------------------------------
>
>                 Key: CALCITE-2012
>                 URL: https://issues.apache.org/jira/browse/CALCITE-2012
>             Project: Calcite
>          Issue Type: Task
>          Components: druid
>    Affects Versions: 1.14.0
>            Reporter: Jesus Camacho Rodriguez
>            Assignee: Jesus Camacho Rodriguez
>             Fix For: 1.15.0
>
>
> CALCITE-1617 introduced LocalInterval as a proper way to close the gap 
> between the semantics of SQL timestamp type and Druid instants.
> After that, CALCITE-1947 introduced 'timestamp with local time zone' type in 
> Calcite and mapped the Druid time column to this type. Thus, we do not need 
> anymore the LocalInterval class and we can use Joda Interval, since the 
> column represents an Instant rather than a LocalDateTime.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to