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