Hello Dan Burkert, Jean-Daniel Cryans, Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10147
to look at the new patch set (#2).
Change subject: [Java] Remove shaded parquet and spark-avro dependencies
......................................................................
[Java] Remove shaded parquet and spark-avro dependencies
Currently the kudu-client-tools project shades parquet and
the kudu-spark-tools project shades spark-avro. According
to commit history this is done for classpath convenience when
the parquet and avro import/export options are used.
However, our current shading configuration in Maven is not
pulling in transitive dependencies but instead only including
the direct classes from those jars. If it works at runtime it's just
luck that the rest of the classes are there or unused.
The Gradle build is currently including all of the transtive
dependencies, but that results in a jar that is very large
including things like the Scala library, Jackson, Snappy, etc.
Instead of packaging/shading such large dependencies,
this patch changes the dependencies to provided scope which
accurately representing the fact that the libarries are expected
to be on the classpath at runtime and document’s the details.
Change-Id: Iccf46be9eebb91e900a9ebb5f99b6510165956e7
---
M docs/developing.adoc
M java/kudu-client-tools/build.gradle
M java/kudu-client-tools/pom.xml
M java/kudu-spark-tools/build.gradle
M java/kudu-spark-tools/pom.xml
5 files changed, 18 insertions(+), 22 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/47/10147/2
--
To view, visit http://gerrit.cloudera.org:8080/10147
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iccf46be9eebb91e900a9ebb5f99b6510165956e7
Gerrit-Change-Number: 10147
Gerrit-PatchSet: 2
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: Jean-Daniel Cryans <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
