Hello Kudu Jenkins, Adar Lieber-Dembo,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/13004

to look at the new patch set (#3).

Change subject: [build] Fix the java compatibility checker script
......................................................................

[build] Fix the java compatibility checker script

This patch fixes check_compatibility.py. Primarily it
changed the existing script to use the Gradle build
in place of the Maven build that no longer exists.

It now uses japicmp in place of java_acc because
java_acc had issues unzipping the jars created by the
kudu build. Additionally japicmp’s filtering seemed to
work better.

The patch also updates the Gradle build to ensure
generated Protobuf classes are annotated with
@Generated to make report filtering easier.

In follow on patches I may incorparate this into the LINT job
and cause failure on API breakage.

Change-Id: Id482da0b3aa5ea35fcfb2b674f0d7ae413045ca3
---
M build-support/check_compatibility.py
M java/gradle/protobuf.gradle
2 files changed, 108 insertions(+), 91 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/04/13004/3
--
To view, visit http://gerrit.cloudera.org:8080/13004
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Id482da0b3aa5ea35fcfb2b674f0d7ae413045ca3
Gerrit-Change-Number: 13004
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Adar Lieber-Dembo <a...@apache.org>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to