Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8250 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/8250 Reviewed-by: Grant Henke <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Dan Burkert <[email protected]> --- M java/gradle.properties M java/gradle/buildscript.gradle M java/gradle/quality.gradle M java/pom.xml 4 files changed, 49 insertions(+), 2 deletions(-) Approvals: Grant Henke: Looks good to me, but someone else must approve Kudu Jenkins: Verified Dan Burkert: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I6ed072138fd42449a477647af4c0cf13b59a5c43 Gerrit-Change-Number: 8250 Gerrit-PatchSet: 4 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins
