Joe McDonnell has uploaded this change for review. (
http://gerrit.cloudera.org:8080/10688
Change subject: Optimize dependencies for Codegen
......................................................................
Optimize dependencies for Codegen
Pieces of the Codegen library (impala-(no-)sse-ir.cc)
depend on Exec, Exprs, Runtime, Udf, and Util. These
extra dependencies prevent Codegen from being built
in parallel to these other components. This extends
the build, especially on systems using high parallelism
such as distcc.
As far as I can tell, these dependencies are not
necessary. This replaces those dependencies with gen-deps.
Testing:
- I forced all targets other than Codegen to wait
5 minutes after gen-deps completes to start
building. This forces Codegen to build without any of
the currently listed dependencies. It built
successfully and I made sure that the following
files were identical to a build without this change:
- be/generated-sources/impala-ir/*
- llvm-ir/*
- I ran the core tests, and they were clean except for
known unrelated issues.
Change-Id: Ie3294cd27c2d35388a04934440a1d2b0ba3a0dd9
---
M be/src/codegen/CMakeLists.txt
1 file changed, 2 insertions(+), 2 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/88/10688/1
--
To view, visit http://gerrit.cloudera.org:8080/10688
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie3294cd27c2d35388a04934440a1d2b0ba3a0dd9
Gerrit-Change-Number: 10688
Gerrit-PatchSet: 1
Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com>
