Matthew Jacobs has uploaded a new patch set (#2). Change subject: IMPALA-5137: pt1, Refactor TimestampValue constructors ......................................................................
IMPALA-5137: pt1, Refactor TimestampValue constructors In preparation for supporting Kudu TIMESTAMPs, some cleanup of the TimestampValue code will be helpful first. This change just refactors some of the TimestampValue constructors which do more than simple construction, e.g. parsing or converting, to static factory methods. Change-Id: Id25e19f7984e5ebf9073d9c569faf69cec142fa1 --- M be/src/benchmarks/parse-timestamp-benchmark.cc M be/src/exec/data-source-scan-node.cc M be/src/exec/text-converter.inline.h M be/src/exprs/aggregate-functions-ir.cc M be/src/exprs/cast-functions-ir.cc M be/src/exprs/decimal-operators-ir.cc M be/src/exprs/expr-test.cc M be/src/exprs/expr-value.h M be/src/exprs/literal.cc M be/src/exprs/timestamp-functions-ir.cc M be/src/exprs/timestamp-functions.cc M be/src/runtime/raw-value-test.cc M be/src/runtime/runtime-state.cc M be/src/runtime/timestamp-test.cc M be/src/runtime/timestamp-value.cc M be/src/runtime/timestamp-value.h M be/src/service/impala-http-handler.cc M be/src/service/impala-server.cc M be/src/service/query-exec-state.cc M be/src/statestore/statestore-subscriber.cc M be/src/util/dict-test.cc 21 files changed, 254 insertions(+), 150 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/10/6510/2 -- To view, visit http://gerrit.cloudera.org:8080/6510 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Id25e19f7984e5ebf9073d9c569faf69cec142fa1 Gerrit-PatchSet: 2 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Matthew Jacobs <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Matthew Jacobs <[email protected]>
