Dan Burkert has posted comments on this change. ( http://gerrit.cloudera.org:8080/10147 )
Change subject: [Java] Remove shaded parquet and spark-avro dependencies ...................................................................... Patch Set 1: Code-Review+1 (1 comment) I'm in favor of this change. This could be considered a breaking change, but I think the kudu-spark-tools package is used by so few people (and these specific features especially so), and the downside of shading these deps is big enough that we should go ahead. http://gerrit.cloudera.org:8080/#/c/10147/1/docs/developing.adoc File docs/developing.adoc: http://gerrit.cloudera.org:8080/#/c/10147/1/docs/developing.adoc@185 PS1, Line 185: kudu-spark-tools add backticks to make it clear this is referring to a specific project here and below (`spark-avro`, `kudu-spark-tools`) -- 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: comment Gerrit-Change-Id: Iccf46be9eebb91e900a9ebb5f99b6510165956e7 Gerrit-Change-Number: 10147 Gerrit-PatchSet: 1 Gerrit-Owner: Grant Henke <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Comment-Date: Mon, 23 Apr 2018 16:18:24 +0000 Gerrit-HasComments: Yes
