impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Matthew Jacobs (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-5539: Fix Kudu timestamp with -use local tz for unix ts
Date Fri, 14 Jul 2017 17:49:31 GMT
Matthew Jacobs has uploaded a new patch set (#4).

Change subject: IMPALA-5539: Fix Kudu timestamp with -use_local_tz_for_unix_ts

IMPALA-5539: Fix Kudu timestamp with -use_local_tz_for_unix_ts

The -use_local_tz_for_unix_timestamp_conversion flag exists
to specify if TIMESTAMPs should be interpreted as localtime
or UTC when converting to/from Unix time via builtins:
  from_unixtime(bigint unixtime)
  unix_timestamp(string datetime[, ...])
  unix_timestamp(timestamp datetime)

However, the KuduScanner was calling into code that, when
the gflag above was set, interpreted Unix times as local
time.  Unfortunately the write path (KuduTableSink) and some
FE TIMESTAMP code (see did not have this
behavior, i.e. we were handling the gflag inconsistently.

* Adds a custom cluster test to run Kudu test cases with
* Adds tests for the new builtin
  unix_micros_to_utc_timestamp() which run in a custom
  cluster test (added as well
  as in the regular tests (added to

Change-Id: I423a810427353be76aa64442044133a9a22cdc9b
M be/src/exec/
M be/src/exprs/
M be/src/exprs/timestamp-functions.h
M be/src/runtime/
M be/src/runtime/timestamp-value.h
M be/src/runtime/timestamp-value.inline.h
M common/function-registry/
M fe/src/main/java/org/apache/impala/catalog/
M fe/src/test/java/org/apache/impala/analysis/
A testdata/workloads/functional-query/queries/QueryTest/utc-timestamp-functions.test
M tests/custom_cluster/
A tests/custom_cluster/
M tests/query_test/
M tests/query_test/
14 files changed, 106 insertions(+), 16 deletions(-)

  git pull ssh:// refs/changes/11/7311/4
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I423a810427353be76aa64442044133a9a22cdc9b
Gerrit-PatchSet: 4
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Matthew Jacobs <>
Gerrit-Reviewer: Dan Hecht <>
Gerrit-Reviewer: Matthew Jacobs <>

View raw message