You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@impala.apache.org by "Tim Armstrong (Code Review)" <ge...@cloudera.org> on 2016/03/04 22:47:20 UTC

[Impala-CR](cdh5-trunk) Cleanup timestamp headers

Tim Armstrong has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/2463

Change subject: Cleanup timestamp headers
......................................................................

Cleanup timestamp headers

The main part of this change is to move timestamp parsing logic from
.h to .cc file. This slightly reduces the size of the llvm bitcode
module and should slightly reduce compile times.

Also slightly change some includes to avoid bringing in unnecessary
headers.

Change-Id: Ib56779cae51503991ae3ecdfe3873edfe474d09f
---
M be/src/exec/hdfs-avro-scanner-ir.cc
M be/src/exprs/timestamp-functions.cc
M be/src/exprs/timestamp-functions.h
M be/src/runtime/buffered-tuple-stream-ir.cc
M be/src/runtime/buffered-tuple-stream.cc
M be/src/runtime/raw-value.h
M be/src/runtime/timestamp-parse-util.cc
M be/src/runtime/timestamp-parse-util.h
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-hs2-server.cc
12 files changed, 476 insertions(+), 425 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib56779cae51503991ae3ecdfe3873edfe474d09f
Gerrit-PatchSet: 1
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Tim Armstrong <ta...@cloudera.com>