> On 2012-02-24 04:49:05, Alejandro Abdelnur wrote: > > /trunk/pom.xml, line 612 > > <https://reviews.apache.org/r/4022/diff/2/?file=85802#file85802line612> > > > > why this new dep is needed? > > Angelo K. Huang wrote: > TestLocalOozieExample will complain NPE if this dep is not there. The > reason might be the changes of minicluster from hadoop jar. I tracked the > deps in oozie-core and found out oozie-core has this dep but not sure where > it gets from. > > > Alejandro Abdelnur wrote: > it seems odd because if run 'mvn dependency:tree' from trunk/examples/ > (first do an mvn install -DskipTests from trunk/) commons-httpclient shows up > > Angelo K. Huang wrote: > Does it show up for oozie-example as well? > > Alejandro Abdelnur wrote: > Not sure what you mean, but here is the output; > > ----- > > [INFO] --- maven-dependency-plugin:2.4:tree (default-cli) @ > oozie-examples --- > [INFO] org.apache.oozie:oozie-examples:jar:3.2.0-SNAPSHOT > [INFO] +- org.apache.oozie:oozie-core:jar:3.2.0-SNAPSHOT:provided > [INFO] | +- org.apache.openjpa:openjpa-persistence:jar:2.1.0:provided > [INFO] | | +- org.apache.openjpa:openjpa-kernel:jar:2.1.0:provided > [INFO] | | | +- org.apache.openjpa:openjpa-lib:jar:2.1.0:provided > [INFO] | | | | \- net.sourceforge.serp:serp:jar:1.13.1:provided > [INFO] | | | +- > org.apache.geronimo.specs:geronimo-jms_1.1_spec:jar:1.1.1:provided > [INFO] | | | \- > org.apache.geronimo.specs:geronimo-jta_1.1_spec:jar:1.1.1:provided > [INFO] | | \- > org.apache.geronimo.specs:geronimo-jpa_2.0_spec:jar:1.1:provided > [INFO] | +- org.apache.openjpa:openjpa-jdbc:jar:2.1.0:provided > [INFO] | +- > org.apache.openjpa:openjpa-persistence-jdbc:jar:2.1.0:provided > [INFO] | +- com.googlecode.json-simple:json-simple:jar:1.1:provided > [INFO] | +- org.jdom:jdom:jar:1.1:provided > [INFO] | +- commons-dbcp:commons-dbcp:jar:1.4:provided > [INFO] | | \- commons-pool:commons-pool:jar:1.5.4:provided > [INFO] | +- javax.servlet:servlet-api:jar:2.5:provided > [INFO] | +- javax.servlet.jsp:jsp-api:jar:2.0:provided > [INFO] | +- javax.mail:mail:jar:1.4:provided > [INFO] | | \- javax.activation:activation:jar:1.1:provided > [INFO] | +- org.apache.hadoop:hadoop-auth:jar:0.23.2-SNAPSHOT:provided > [INFO] | \- postgresql:postgresql:jar:9.0-801.jdbc4:provided > [INFO] +- org.apache.oozie:oozie-core:jar:tests:3.2.0-SNAPSHOT:test > [INFO] +- org.apache.oozie:oozie-hadoop:jar:1.0.1:provided > [INFO] | \- org.apache.hadoop:hadoop-client:jar:1.0.1:provided > [INFO] +- org.apache.oozie:oozie-hadoop-test:jar:1.0.1:test > [INFO] | \- org.apache.hadoop:hadoop-minicluster:jar:1.0.1:test > [INFO] | +- org.apache.hadoop:hadoop-test:jar:1.0.1:test > [INFO] | | +- org.apache.ftpserver:ftplet-api:jar:1.0.0:test > [INFO] | | +- org.apache.mina:mina-core:jar:2.0.0-M5:test > [INFO] | | +- org.apache.ftpserver:ftpserver-core:jar:1.0.0:test > [INFO] | | \- > org.apache.ftpserver:ftpserver-deprecated:jar:1.0.0-M2:test > [INFO] | \- com.sun.jersey:jersey-server:jar:1.0:test > [INFO] | \- com.sun.jersey:jersey-core:jar:1.0:test > [INFO] | \- javax.ws.rs:jsr311-api:jar:1.0:test > [INFO] +- org.apache.oozie:oozie-client:jar:3.2.0-SNAPSHOT:provided > [INFO] +- org.apache.oozie:oozie-sharelib-pig:jar:3.2.0-SNAPSHOT:compile > [INFO] | +- org.apache.pig:pig:jar:0.9.0:compile > [INFO] | | +- commons-cli:commons-cli:jar:1.2:compile > [INFO] | | +- xmlenc:xmlenc:jar:0.52:compile > [INFO] | | +- commons-codec:commons-codec:jar:1.4:compile > [INFO] | | +- commons-net:commons-net:jar:1.4.1:compile > [INFO] | | \- org.apache.hadoop:avro:jar:1.3.2:compile > [INFO] | | +- > org.codehaus.jackson:jackson-mapper-asl:jar:1.0.1:compile > [INFO] | | | \- > org.codehaus.jackson:jackson-core-asl:jar:1.4.2:compile > [INFO] | | +- org.slf4j:slf4j-api:jar:1.5.8:compile > [INFO] | | +- com.thoughtworks.paranamer:paranamer:jar:2.2:compile > [INFO] | | +- > com.thoughtworks.paranamer:paranamer-ant:jar:2.2:compile > [INFO] | | | \- > com.thoughtworks.paranamer:paranamer-generator:jar:2.2:compile > [INFO] | | | +- com.thoughtworks.qdox:qdox:jar:1.10.1:compile > [INFO] | | | \- asm:asm:jar:3.1:compile > [INFO] | | \- commons-lang:commons-lang:jar:2.4:compile > [INFO] | +- jline:jline:jar:0.9.94:compile > [INFO] | +- org.python:jython:jar:2.5.0:compile > [INFO] | +- org.antlr:antlr-runtime:jar:3.4:compile > [INFO] | | +- org.antlr:stringtemplate:jar:3.2.1:compile > [INFO] | | \- antlr:antlr:jar:2.7.7:compile > [INFO] | \- com.google.guava:guava:jar:r09:compile > [INFO] +- > org.apache.oozie:oozie-sharelib-streaming:jar:3.2.0-SNAPSHOT:compile > [INFO] | \- org.apache.hadoop:hadoop-streaming:jar:1.0.1:compile > [INFO] | \- org.apache.hadoop:hadoop-core:jar:1.0.1:compile > [INFO] | +- commons-httpclient:commons-httpclient:jar:3.0.1:compile > [INFO] | | \- commons-logging:commons-logging:jar:1.1:compile > [INFO] | | \- log4j:log4j:jar:1.2.15:compile > [INFO] | +- org.apache.commons:commons-math:jar:2.1:compile > [INFO] | +- > commons-configuration:commons-configuration:jar:1.6:compile > [INFO] | | +- > commons-collections:commons-collections:jar:3.2.1:compile > [INFO] | | +- commons-digester:commons-digester:jar:1.8:compile > [INFO] | | | \- > commons-beanutils:commons-beanutils:jar:1.7.0:compile > [INFO] | | \- > commons-beanutils:commons-beanutils-core:jar:1.8.0:compile > [INFO] | +- org.mortbay.jetty:jetty:jar:6.1.14:compile > [INFO] | | \- org.mortbay.jetty:servlet-api-2.5:jar:6.1.14:compile > [INFO] | +- org.mortbay.jetty:jetty-util:jar:6.1.26:compile > [INFO] | +- tomcat:jasper-runtime:jar:5.5.12:compile > [INFO] | +- tomcat:jasper-compiler:jar:5.5.12:compile > [INFO] | +- org.mortbay.jetty:jsp-api-2.1:jar:6.1.14:compile > [INFO] | +- org.mortbay.jetty:jsp-2.1:jar:6.1.14:compile > [INFO] | | \- ant:ant:jar:1.6.5:compile > [INFO] | +- commons-el:commons-el:jar:1.0:compile > [INFO] | +- net.java.dev.jets3t:jets3t:jar:0.7.1:compile > [INFO] | +- net.sf.kosmosfs:kfs:jar:0.3:compile > [INFO] | +- oro:oro:jar:2.0.8:compile > [INFO] | \- org.eclipse.jdt:core:jar:3.1.1:compile > [INFO] +- org.slf4j:slf4j-log4j12:jar:1.5.8:test > [INFO] +- junit:junit:jar:3.8.1:test (scope not updated to compile) > [INFO] +- org.apache.derby:derby:jar:10.6.1.0:compile > [INFO] \- hsqldb:hsqldb:jar:1.8.0.7:compile > [INFO] > ------------------------------------------------------------------------ > ---- > > Angelo K. Huang wrote: > yes, but commons-httpclient is not under oozie-examples. > > Alejandro Abdelnur wrote: > * commons-httpclient is not used by oozie-examples, thus it should not be > an explicit dependency of it. > * commons-httpclient ends up in the testclasspath regardless how pulls it. > > What am I missing?
oozie-examples test case need commons-httpclient, otherwise the test case will fail. you can try it on linux with -Dtest=TestLocalOozieExample. - Angelo K. ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4022/#review5319 ----------------------------------------------------------- On 2012-02-24 02:54:07, Angelo K. Huang wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4022/ > ----------------------------------------------------------- > > (Updated 2012-02-24 02:54:07) > > > Review request for oozie. > > > Summary > ------- > > TestLocalOozieExample failed when running on RH linux. The test case needs to > use XFsTestCase. > > > This addresses bug OOZIE-715. > https://issues.apache.org/jira/browse/OOZIE-715 > > > Diffs > ----- > > /trunk/examples/pom.xml 1292604 > > /trunk/examples/src/test/java/org/apache/oozie/example/TestLocalOozieExample.java > 1292604 > /trunk/pom.xml 1292604 > > Diff: https://reviews.apache.org/r/4022/diff > > > Testing > ------- > > > Thanks, > > Angelo K. > >
