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

Reply via email to