Grant Henke has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/15360 )

Change subject: [build] Use Gradle to build the hms-plugin jar
......................................................................

[build] Use Gradle to build the hms-plugin jar

In an effort to unify how we build Java from cmake/make, this
patch converts the `hms-plugin.jar` compilation from using
`add_jar` to using `add_custom_command` calling a Gradle
command.

In order to ensure mutliple calls to Gradle don’t try and
download the `gradle-wrapper.jar` at the same time, I added an
`init-gradle` target that all Gradle commands should depend on.

I also added `--quiet --console=plain` command line flags to
the Gradle commands to prevent weird Gradle logging within the
make build.

Change-Id: I3312b1605c4f050380264688f9d1b78038608257
Reviewed-on: http://gerrit.cloudera.org:8080/15360
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <[email protected]>
---
M CMakeLists.txt
M build-support/run_dist_test.py
M src/kudu/hms/CMakeLists.txt
M src/kudu/subprocess/CMakeLists.txt
4 files changed, 36 insertions(+), 18 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Adar Dembo: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I3312b1605c4f050380264688f9d1b78038608257
Gerrit-Change-Number: 15360
Gerrit-PatchSet: 6
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to