Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10425 )
Change subject: [java] Add kudu-client dep to kudu-spark-tools ...................................................................... [java] Add kudu-client dep to kudu-spark-tools We use the kudu-client APIs in kudu-spark-tools but don’t define it as an explicit dependency. Using transitive dependencies is a bad practice but generally doesn’t break anything. However, the Gradle Intellij integration seams to not find the kudu-client project dependnecy if it’s not explicitly listed here. Change-Id: I6255bedac9781981ae8a937f0dc72afff0fc5ef0 Reviewed-on: http://gerrit.cloudera.org:8080/10425 Tested-by: Grant Henke <[email protected]> Reviewed-by: Adar Dembo <[email protected]> --- M java/kudu-spark-tools/build.gradle M java/kudu-spark-tools/pom.xml 2 files changed, 6 insertions(+), 0 deletions(-) Approvals: Grant Henke: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/10425 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I6255bedac9781981ae8a937f0dc72afff0fc5ef0 Gerrit-Change-Number: 10425 Gerrit-PatchSet: 2 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]>
