You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jira@arrow.apache.org by "Joris Van den Bossche (Jira)" <ji...@apache.org> on 2021/05/18 10:56:00 UTC

[jira] [Updated] (ARROW-12147) TimestampTZ functions

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

Joris Van den Bossche updated ARROW-12147:
------------------------------------------
    Labels: timezone  (was: )

> TimestampTZ functions
> ---------------------
>
>                 Key: ARROW-12147
>                 URL: https://issues.apache.org/jira/browse/ARROW-12147
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++ - Gandiva
>            Reporter: Yuan Zhou
>            Priority: Major
>              Labels: timezone
>
> hi arrow developers, 
> Gandiva supports timestamp related functions already - but it looks like UTC only. It would be nice to support TimestampTZ(timestamp with timezone) also. Pass timestamp w/ timezone to those functions will return wrong result(should consider the timezone offset)
> [https://github.com/apache/arrow/blob/master/cpp/src/gandiva/precompiled/time.cc#L41]
> I guess we could provide a helper func like below to convert to timestamp w/ UTC first, then all existing functions should be working correctly. 
> {{_ConvertTIMESTAMP(timestamp, timezone)_}}
> A better way may require re-implement those functions by considering the zone offset when doing calculating, but this may make the code looks complicated. 
> Note, castTIMESTAMP_utf8 supports creating timestamp with timezone already. 
> [https://github.com/apache/arrow/blob/master/cpp/src/gandiva/precompiled/time.cc#L618-L743]
>  
> thanks, -yuan



--
This message was sent by Atlassian Jira
(v8.3.4#803005)