Yuan Zhou created ARROW-12147: --------------------------------- Summary: 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
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)