Hello Dan Burkert, Kudu Jenkins, Grant Henke,

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

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

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

Change subject: java-client: enforce that only Java 7 APIs are used
......................................................................

java-client: enforce that only Java 7 APIs are used

The underlying enforcement machinery is powered by animal-sniffer, which has
both maven and gradle plugins. The maven plugin doesn't check test code [1],
but the gradle plugin does.

To test this, I added a Process.destroyForcibly() call to MiniKuduCluster,
which caused the gradle build to fail. I also added a
ConcurrentHashMap.keySet() call to Statistics (i.e. KUDU-2188), which caused
both builds to fail.

1. https://github.com/mojohaus/animal-sniffer/issues/5

Change-Id: I6ed072138fd42449a477647af4c0cf13b59a5c43
---
M java/gradle/buildscript.gradle
M java/gradle/quality.gradle
M java/pom.xml
3 files changed, 38 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/50/8250/2
--
To view, visit http://gerrit.cloudera.org:8080/8250
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I6ed072138fd42449a477647af4c0cf13b59a5c43
Gerrit-Change-Number: 8250
Gerrit-PatchSet: 2
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins

Reply via email to