Grant Henke has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10096 )

Change subject: [Java] Fix Guava transitive dependecies
......................................................................


Patch Set 3:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/10096/3//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/10096/3//COMMIT_MSG@16
PS3, Line 16: d’s
> Adds
Done


http://gerrit.cloudera.org:8080/#/c/10096/3/java/kudu-client-tools/build.gradle
File java/kudu-client-tools/build.gradle:

http://gerrit.cloudera.org:8080/#/c/10096/3/java/kudu-client-tools/build.gradle@24
PS3, Line 24:   compile(libs.guava) { transitive = false }
> Is there a way to factor this out into a common configuration across all su
I can't set "transitive = false" globally without a some really 
hacky/experimental and incubating api usage, but I can explicitly exclude all 
of the annotations dependencies explicitly globally. It's probably better to be 
more explicit anyway.



--
To view, visit http://gerrit.cloudera.org:8080/10096
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8f1f6e762629c1dea30589a1ade1f3da6de6ad3d
Gerrit-Change-Number: 10096
Gerrit-PatchSet: 3
Gerrit-Owner: Grant Henke <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Dan Burkert <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Sat, 21 Apr 2018 15:20:15 +0000
Gerrit-HasComments: Yes

Reply via email to