Jason Fehr has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/21203


Change subject: IMPALA-12426: Switches the duration fields to be stored in 
decimal seconds.
......................................................................

IMPALA-12426: Switches the duration fields to be stored in decimal seconds.

The original implementation of the completed queries table
stored durations in integer nanoseconds and provided the
prettyprint_duration SQL function to format the values into
human readable form.

This change modifies the duration fields to be stored as
seconds with up to three digits of millisecond precision.

The prettyprint_duration function would have required a
large investment of time to make it work with decimal
values, and since the new format is more human readable
anyways, this function has been removed.

Testing was accomplished by modifying the python custom
cluster tests.

Change-Id: I842951a132b7b8eadccb09a3674f4c34ac42ff1b
---
M be/src/benchmarks/expr-benchmark.cc
M be/src/exprs/expr-test.cc
M be/src/exprs/string-functions-ir.cc
M be/src/exprs/string-functions.h
M be/src/service/workload-management-fields.cc
M be/src/service/workload-management.cc
M be/src/service/workload-management.h
M common/function-registry/impala_functions.py
M docs/topics/impala_functions.xml
M docs/topics/impala_string_functions.xml
D 
testdata/workloads/functional-query/queries/QueryTest/prettyprint-duration.test
D tests/query_test/test_prettyprint_duration.py
M tests/util/assert_time.py
M tests/util/workload_management.py
14 files changed, 124 insertions(+), 352 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/03/21203/2
--
To view, visit http://gerrit.cloudera.org:8080/21203
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I842951a132b7b8eadccb09a3674f4c34ac42ff1b
Gerrit-Change-Number: 21203
Gerrit-PatchSet: 2
Gerrit-Owner: Jason Fehr <jf...@cloudera.com>
Gerrit-Reviewer: Andrew Sherman <asher...@cloudera.com>
Gerrit-Reviewer: Jason Fehr <jf...@cloudera.com>
Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>

Reply via email to