Hello Yingchun Lai, Attila Bukor, Kudu Jenkins, Abhishek Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/20307
to look at the new patch set (#2).
Change subject: WIP [build] introduce NO_JAVA option
......................................................................
WIP [build] introduce NO_JAVA option
After introduction of hms_plugin.jar and subprocess.jar as dependencies
for kudu-master, it's become impossible to build all the rest of the
project's C++ components when Java and Gradle wasn't present on a build
machine. However, sometimes it's necessary to build just Kudu C++
client library or a set of benchmark tests that don't use any of the
functionality dependent on the HMS integration or Ranger fine-grained
authz enforcement. In addition, it was nice to have an ability to work
on Kudu's C++ code even without Internet access when all the 3rd-party
components had already been built, but it also become impossible once
the HMS integration had been introduced.
This patch addresses the issues mentioned above, adding a new NO_JAVA
configuration option for cmake: it's not enabled by default, of course.
I also updated the build-support/dist_test.py script to be able
to submit the result bits built with NO_JAVA option for distributed
testing without hms_plgin.jar and subprocess.jar being present. This
patch is also useful to reduce the compilation time when building tests
for micro-benchmarking: for details see src/kudu/scripts/benchmarks.sh.
WIP:
* get initial feedback
* make sure all the C++ tests that are built pass in NO_JAVA builds
* brush up the corresponding code in dist_test.py to skip searching
and uploading dependencies that aren't present in NO_JAVA builds
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/integration-tests/tablet_copy-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/postgres/CMakeLists.txt
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
20 files changed, 193 insertions(+), 78 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/07/20307/2
--
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: newpatchset
Gerrit-Change-Id: I3d59caa3bc718a2f53a798c1e97e367d9f65c5ae
Gerrit-Change-Number: 20307
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yingchun Lai <[email protected]>
