Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/8250 )
Change subject: java-client: enforce that only Java 7 APIs are used ...................................................................... Patch Set 2: Code-Review+1 (2 comments) Looks good to me. Just a couple comments on things you can choose to address/reply-to or punt on. http://gerrit.cloudera.org:8080/#/c/8250/2//COMMIT_MSG Commit Message: http://gerrit.cloudera.org:8080/#/c/8250/2//COMMIT_MSG@10 PS2, Line 10: both maven and gradle plugins. The maven plugin doesn't check test code [1], Do we need to check test code? Wouldn't it be okay to write test code that leverages Java 8 APIs. http://gerrit.cloudera.org:8080/#/c/8250/2/java/gradle/quality.gradle File java/gradle/quality.gradle: http://gerrit.cloudera.org:8080/#/c/8250/2/java/gradle/quality.gradle@74 PS2, Line 74: signature 'org.codehaus.mojo.signature:java17:1.0@signature' Is their some ay we could leverage the "javaCompatibility" value in gradle.properties to set this? -- 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: comment 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 Gerrit-Comment-Date: Thu, 12 Oct 2017 01:54:09 +0000 Gerrit-HasComments: Yes
