Grant Henke has posted comments on this change. ( http://gerrit.cloudera.org:8080/13691 )
Change subject: [backup] Fix backup tools packaging ...................................................................... Patch Set 3: (1 comment) http://gerrit.cloudera.org:8080/#/c/13691/3/java/kudu-backup-common/build.gradle File java/kudu-backup-common/build.gradle: http://gerrit.cloudera.org:8080/#/c/13691/3/java/kudu-backup-common/build.gradle@31 PS3, Line 31: provided libs.scalaLibrary > Is this change to make scala provided required or just cleanup? Just wonder It's not "required" per-se, but it's more correct given we expect the tools and backup modules to specify it. -- To view, visit http://gerrit.cloudera.org:8080/13691 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I334a38bb5d7252b5e6d2722c4f1fc1d3b5622db7 Gerrit-Change-Number: 13691 Gerrit-PatchSet: 3 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-Comment-Date: Fri, 21 Jun 2019 16:38:56 +0000 Gerrit-HasComments: Yes
