Internal Jenkins has submitted this change and it was merged. Change subject: IMPALA-1112: Remove some unncessary code from cross-compilation ......................................................................
IMPALA-1112: Remove some unncessary code from cross-compilation This change stops including some boost library header files which pulls in other unnecessary boost library header files. This reduces the amount of cross-compiled code which needs to be materialized during codegen. This change also removes some UDF's Prepare() and Close() functions and UDF functions fromUtc(), toUtc() and uuid() from cross-compilation as they won't benefit from it. With this change, the bitcode module reduces from 2.12 MB to 1.86MB. Change-Id: I543809c69da0b4085a0e299b91cd550b274c46af Reviewed-on: http://gerrit.cloudera.org:8080/3793 Reviewed-by: Michael Ho <[email protected]> Tested-by: Internal Jenkins --- M be/src/benchmarks/parse-timestamp-benchmark.cc M be/src/common/init.cc M be/src/exprs/CMakeLists.txt M be/src/exprs/expr.cc M be/src/exprs/timestamp-functions-ir.cc A be/src/exprs/timestamp-functions.cc M be/src/exprs/timestamp-functions.h M be/src/exprs/timezone_db.cc A be/src/exprs/timezone_db.h M be/src/exprs/utility-functions-ir.cc A be/src/exprs/utility-functions.cc M be/src/exprs/utility-functions.h M be/src/runtime/mem-tracker.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/statestore/statestore.cc M be/src/util/debug-util.cc 18 files changed, 331 insertions(+), 212 deletions(-) Approvals: Michael Ho: Looks good to me, approved Internal Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/3793 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I543809c69da0b4085a0e299b91cd550b274c46af Gerrit-PatchSet: 5 Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-Owner: Michael Ho <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Internal Jenkins Gerrit-Reviewer: Michael Ho <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
