> 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
Does it show up for oozie-example as well? - 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. > >
