Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/10059 )
Change subject: [Java] Add Yetus doclet to Gradle build ...................................................................... Patch Set 1: (2 comments) http://gerrit.cloudera.org:8080/#/c/10059/1/java/gradle/compile.gradle File java/gradle/compile.gradle: http://gerrit.cloudera.org:8080/#/c/10059/1/java/gradle/compile.gradle@a28 PS1, Line 28: > Is this implicitly defined in Yetus already? Well Yetus does this weird thing where it filters out doclint warnings. That's why we didn't need this in the Maven build. I opened YETUS-626 to track that when removing this. http://gerrit.cloudera.org:8080/#/c/10059/1/java/gradle/docs.gradle File java/gradle/docs.gradle: http://gerrit.cloudera.org:8080/#/c/10059/1/java/gradle/docs.gradle@34 PS1, Line 34: 7 > Should this be 8? We still support Java 7 so I link to that. -- To view, visit http://gerrit.cloudera.org:8080/10059 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I283a3ccb8003356c875c603c9169d68a66b45804 Gerrit-Change-Number: 10059 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: Grant Henke <granthe...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Fri, 13 Apr 2018 17:34:40 +0000 Gerrit-HasComments: Yes