You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Jesus Camacho Rodriguez (JIRA)" <ji...@apache.org> on 2017/11/14 23:45:00 UTC

[jira] [Resolved] (CALCITE-2012) Replace LocalInterval by Interval in Druid adapter

     [ https://issues.apache.org/jira/browse/CALCITE-2012?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Jesus Camacho Rodriguez resolved CALCITE-2012.
----------------------------------------------
    Resolution: Fixed

[~julianhyde], patch was in fact ready.

Fixed in http://git-wip-us.apache.org/repos/asf/calcite/commit/20ade9d.

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