Impala Public Jenkins has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/13775 )

Change subject: build: use thin static archives
......................................................................

build: use thin static archives

This enables thin static archives for our internal libraries during the
build. This makes linking much faster since the static archives just
point to object files instead of copying them.

This reduces the size of intermediate '.a' files for my debug build from
about 1.4GB to 58MB.

Incremental rebuilds are slightly sped up, though maybe in the realm of
noise (likely the performance improvement depends on how much RAM is
available for buffer caching the IO):

Without patch incremental build of catalogd after modifying 
CatalogObjects.thrift:
real    0m53.433s
user    7m6.400s
sys     1m21.610s

With patch:
real    0m44.772s
user    7m6.632s
sys     1m16.870s

Change-Id: I3b54b5be8658c951914758c406cca55d4cc1756e
Reviewed-on: http://gerrit.cloudera.org:8080/13775
Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Tested-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
---
M CMakeLists.txt
1 file changed, 9 insertions(+), 0 deletions(-)

Approvals:
  Impala Public Jenkins: Looks good to me, approved; Verified

--
To view, visit http://gerrit.cloudera.org:8080/13775
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I3b54b5be8658c951914758c406cca55d4cc1756e
Gerrit-Change-Number: 13775
Gerrit-PatchSet: 4
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to