Skip to content
This repository has been archived by the owner on Apr 5, 2022. It is now read-only.

[Sprint: 50] XD-3148 Adding exclusion to remove hadoop-core-2.5.0-mr1-cdh5.3.3.jar… #1689

Closed
wants to merge 1 commit into from

Conversation

trisberg
Copy link
Contributor

@trisberg trisberg commented Jun 4, 2015

… from dist

@sabbyanandan sabbyanandan changed the title XD-3148 Adding exclusion to remove hadoop-core-2.5.0-mr1-cdh5.3.3.jar… [Sprint: 50] XD-3148 Adding exclusion to remove hadoop-core-2.5.0-mr1-cdh5.3.3.jar… Jun 5, 2015
@jvalkeal
Copy link
Contributor

jvalkeal commented Jun 5, 2015

Oh dear Thomas, how on earth I'm supposed to test this?

@trisberg
Copy link
Contributor Author

trisberg commented Jun 5, 2015

Just build the distribution zip files and compare the xd/lib/cdh5 directory content with what is in 1.2.0.RC1

@jvalkeal
Copy link
Contributor

jvalkeal commented Jun 8, 2015

looks ok, merging, just adding comment here for jetty-util

This patch changes that

$ diff /tmp/cdh.1 /tmp/cdh.2
8d7
< hadoop-core-2.5.0-mr1-cdh5.3.3.jar
28c27
< jetty-util-6.1.26.jar
---
> jetty-util-6.1.26.cloudera.4.jar

We seem to have jetty-util also directly under lib.

$ find . |grep jetty-util
./phd21/jetty-util-6.1.26.jar
./phd21/jetty-util-7.6.10.v20130312.jar
./jetty-util-6.1.26.jar
./hdp22/jetty-util-6.1.26.jar
./jetty-util-8.1.14.v20131031.jar
./hadoop26/jetty-util-6.1.26.jar
./cdh5/jetty-util-6.1.26.cloudera.4.jar
./phd30/jetty-util-6.1.26.jar

@jvalkeal
Copy link
Contributor

jvalkeal commented Jun 8, 2015

merged

@jvalkeal jvalkeal closed this Jun 8, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
2 participants