Adar Dembo has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10074 )

Change subject: [java] Use gradle caching now that it is stable in 4.6
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/10074/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/10074/1//COMMIT_MSG@7
PS1, Line 7: [java] Use gradle caching now that it is stable in 4.6
> If you think there is risk in the space being used, I am happy to leave it
We discussed this over Slack but I'll restate my position here for the benefit 
of others: until we can convince ourselves that the build time savings are 
significant, I think we should ensure this is opt-in like ccache.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ibf5cc3c7a1263b085511c825118fb814e1b65008
Gerrit-Change-Number: 10074
Gerrit-PatchSet: 2
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: Mon, 16 Apr 2018 20:19:21 +0000
Gerrit-HasComments: Yes

Reply via email to