Marton Greber has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/23407 )

Change subject: KUDU-3697: Bundle kudu-spark classes
......................................................................

KUDU-3697: Bundle kudu-spark classes

Update the kudu-spark dependency configuration to implementation to
ensure its classes are bundled into the kudu-backup and
kudu-spark-tools JARs.

Manually tweak the Gradle publishing process so that this dependency
is declared as a compile-time dependency. There are likely better ways
to address this, hence the TODO.

This restores the previous behavior from before the Gradle upgrade:
bundling kudu-spark into the subprojects while keeping the dependency
scope as compile.

Change-Id: I8d3e3c95efe161adc916332477f8cd9f35c48e38
Reviewed-on: http://gerrit.cloudera.org:8080/23407
Tested-by: Abhishek Chennaka <[email protected]>
Reviewed-by: Zoltan Chovan <[email protected]>
Reviewed-by: Marton Greber <[email protected]>
---
M java/gradle/publishing.gradle
M java/kudu-backup/build.gradle
M java/kudu-spark-tools/build.gradle
3 files changed, 8 insertions(+), 2 deletions(-)

Approvals:
  Abhishek Chennaka: Verified
  Zoltan Chovan: Looks good to me, but someone else must approve
  Marton Greber: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I8d3e3c95efe161adc916332477f8cd9f35c48e38
Gerrit-Change-Number: 23407
Gerrit-PatchSet: 2
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Zoltan Chovan <[email protected]>

Reply via email to