Adar Dembo has posted comments on this change.

Change subject: KUDU-2066. Add experimental Gradle build support
......................................................................


Patch Set 14:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/7258/17/java/build.gradle
File java/build.gradle:

Line 19: buildscript { apply from: file('gradle/buildscript.gradle'), to: 
buildscript }
Got trailing whitespace here.


http://gerrit.cloudera.org:8080/#/c/7258/14/java/gradle/dependencies.gradle
File java/gradle/dependencies.gradle:

Line 60: versions["scalaBase"] = versions.scala.substring(0, 
versions.scala.lastIndexOf("."))
> Oh sorry meant to respond. If the Scala version doesn't match this expected
Sure.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib257cdd019a1f383c886b9238bb47d96576c4421
Gerrit-PatchSet: 14
Gerrit-Project: kudu
Gerrit-Branch: master
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-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: Yes

Reply via email to