Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16953 )
Change subject: [build] Separate protobuf java compilation into a seperate module ...................................................................... [build] Separate protobuf java compilation into a seperate module Currently we compile the protobufs in both the kudu-client and the kudu-subprocess. Doing this also combines them onto the same classpath as the module itself. This patch breaks those classes into a separate `kudu-proto` module so that we can avoid duplicate work and ignore the `kudu-proto` module in API compatibility and test coverage checks. This module is not published, so there is no net change to the final artifacts. Change-Id: Ib83db74c4a09cdd597bee90f5124ed804adac42d Reviewed-on: http://gerrit.cloudera.org:8080/16953 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> --- M build-support/check_compatibility.py M java/kudu-client/build.gradle A java/kudu-proto/build.gradle M java/kudu-subprocess/build.gradle M java/settings.gradle 5 files changed, 46 insertions(+), 26 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/16953 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ib83db74c4a09cdd597bee90f5124ed804adac42d Gerrit-Change-Number: 16953 Gerrit-PatchSet: 4 Gerrit-Owner: Grant Henke <[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: Kudu Jenkins (120)
