Csaba Ringhofer has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/17412


Change subject: IMPALA-5121: Fix AVG() on timestamp col with 
use_local_tz_for_unix_timestamp_conversions
......................................................................

IMPALA-5121: Fix AVG() on timestamp col with 
use_local_tz_for_unix_timestamp_conversions

AVG used to contain a back and forth timezone conversion if
use_local_tz_for_unix_timestamp_conversions is true. This could
affect the results if there were values from different DST rules.

Note that AVG on timestamps has other issues besides this, see
IMPALA-7472 for details.

Testing:
- added a regression test

Change-Id: I999099de8e07269b96b75d473f5753be4479cecd
---
M be/src/exprs/aggregate-functions-ir.cc
M 
testdata/workloads/functional-query/queries/QueryTest/utc-timestamp-functions.test
2 files changed, 15 insertions(+), 6 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/12/17412/1
--
To view, visit http://gerrit.cloudera.org:8080/17412
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I999099de8e07269b96b75d473f5753be4479cecd
Gerrit-Change-Number: 17412
Gerrit-PatchSet: 1
Gerrit-Owner: Csaba Ringhofer <csringho...@cloudera.com>

Reply via email to