Matthew Jacobs has uploaded a new change for review. http://gerrit.cloudera.org:8080/6510
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/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/util/dict-test.cc 18 files changed, 229 insertions(+), 129 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/10/6510/1 -- To view, visit http://gerrit.cloudera.org:8080/6510 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Id25e19f7984e5ebf9073d9c569faf69cec142fa1 Gerrit-PatchSet: 1 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Matthew Jacobs <[email protected]>
