You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Grant Henke (Code Review)" <ge...@cloudera.org> on 2017/06/27 16:23:50 UTC

[kudu-CR] [java] Ensure the shaded packages are alway in the same location

Grant Henke has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/7302

Change subject: [java] Ensure the shaded packages are alway in the same location
......................................................................

[java] Ensure the shaded packages are alway in the same location

Currently when a module depends on kudu-clients and is
shaded it is possible to get bloat in the jar due to
dependencies like guava being included in the shaded
jar twice.

This moves the shade plugin and reolocation configuration
to the parent poms pluginManagement section to ensure the
same packages are always placed in the same location.

For example this reduces the kudu-client-tools jar
from 13.4 MB to 8.5 MB.

Note: The relocation target is changed from
org.apache.kudu.clients.shaded to
org.apache.kudu.shaded since the shading is not specific to
the clients anymore.

Change-Id: Ib830ba386313763ad96511dc19a65beb3bb24543
---
M java/kudu-client-tools/pom.xml
M java/kudu-client/pom.xml
M java/kudu-flume-sink/pom.xml
M java/kudu-spark-tools/pom.xml
M java/kudu-spark/pom.xml
M java/pom.xml
6 files changed, 38 insertions(+), 64 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/02/7302/1
-- 
To view, visit http://gerrit.cloudera.org:8080/7302
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib830ba386313763ad96511dc19a65beb3bb24543
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Grant Henke <gr...@gmail.com>

[kudu-CR] [java] Ensure the shaded packages are alway in the same location

Posted by "Grant Henke (Code Review)" <ge...@cloudera.org>.
Grant Henke has abandoned this change.

Change subject: [java] Ensure the shaded packages are alway in the same location
......................................................................


Abandoned

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

Gerrit-MessageType: abandon
Gerrit-Change-Id: Ib830ba386313763ad96511dc19a65beb3bb24543
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Grant Henke <gr...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins