Alexey Serbin has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/20307


Change subject: WIP [build] introduce NO_JAVA option
......................................................................

WIP [build] introduce NO_JAVA option

I found it's pretty annoying to see gradle trying to fetch some Java
packages when I'm trying to build and test C++, not Java components of
the Kudu codebase.  That happens even if all the thirdparty components
are already built and installed.  In particular, various benchmark
tests don't need any Java-related components.

That came out when I was trying to build Kudu on a machine that didn't
have the Internet access.  I recall it was possible to work on Kudu on a
machine not connected to the Internet long time ago, but it seems that's
no longer an option once the HMS integration has been introduced.

This patch addresses the issue, introducing a new NO_JAVA configuration
option for cmake (it's not enabled by default, of course).  I also
updated the dist-test script to be able to submit the result bits for
distributed testing once the build machine with built Kudu binaries
and tests is connected to the internet for a moment.  This patch is
also useful to reduce the compilation time when building tests for
micro-benchmarking (see src/kudu/scripts/benchmarks.sh).

Change-Id: I3d59caa3bc718a2f53a798c1e97e367d9f65c5ae
---
M CMakeLists.txt
M build-support/dist_test.py
M src/kudu/hms/CMakeLists.txt
M src/kudu/integration-tests/CMakeLists.txt
M src/kudu/integration-tests/alter_table-randomized-test.cc
M src/kudu/integration-tests/master-stress-test.cc
M src/kudu/integration-tests/master_failover-itest.cc
M src/kudu/integration-tests/security-itest.cc
M src/kudu/master/CMakeLists.txt
M src/kudu/mini-cluster/CMakeLists.txt
M src/kudu/mini-cluster/external_mini_cluster-test.cc
M src/kudu/mini-cluster/external_mini_cluster.cc
M src/kudu/mini-cluster/external_mini_cluster.h
M src/kudu/ranger-kms/CMakeLists.txt
M src/kudu/ranger/CMakeLists.txt
M src/kudu/subprocess/CMakeLists.txt
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_test.cc
18 files changed, 148 insertions(+), 50 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/07/20307/1
--
To view, visit http://gerrit.cloudera.org:8080/20307
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I3d59caa3bc718a2f53a798c1e97e367d9f65c5ae
Gerrit-Change-Number: 20307
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>

Reply via email to