> On 2012-02-03 03:11:48, Mohammad Islam wrote: > > trunk/pom.xml, line 137 > > <https://reviews.apache.org/r/3726/diff/3/?file=72016#file72016line137> > > > > why do we need it?
we don't. removing. - Alejandro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3726/#review4786 ----------------------------------------------------------- On 2012-02-02 22:37:20, Alejandro Abdelnur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3726/ > ----------------------------------------------------------- > > (Updated 2012-02-02 22:37:20) > > > Review request for oozie. > > > Summary > ------- > > The current mechanism allow to easily package/use new versions of hadoop > without complicating Oozie's POMs. > > mr1 & mr2 profiles are gone from oozie, now it refers to one of the versions > in hadooplibs. > > New maven modules under hadooplib define the hadoop-client/hadoop-test POMs > for different hadoop versions. > > Note that because of HADOOP-8009, hadoop will start providing a hadoop-client > artifact (even for already released versions), still we'll need the > corresponding hadooplibs module to be able to use the assembly (as it is done > as part of this patch) to pull into the oozie distro the JARs for the > supported/tested versions of Hadoop required for the client side. > > Note that this can be used tosimplify the logic of addtowar.sh that won't > have to be aware of the JARs deps of different versions of Hadoop or of > hadoop JARs at all. > > > This addresses bug OOZIE-667. > https://issues.apache.org/jira/browse/OOZIE-667 > > > Diffs > ----- > > trunk/core/pom.xml 1239887 > trunk/core/src/test/java/org/apache/oozie/test/XTestCase.java 1239887 > trunk/docs/src/site/twiki/AG_Install.twiki 1239887 > trunk/docs/src/site/twiki/DG_QuickStart.twiki 1239887 > trunk/examples/pom.xml 1239887 > trunk/hadooplibs/hadoop-0_23_1/pom.xml PRE-CREATION > trunk/hadooplibs/hadoop-0_24_0/pom.xml PRE-CREATION > trunk/hadooplibs/hadoop-1_0_0/pom.xml PRE-CREATION > trunk/hadooplibs/hadoop-test-0_23_1/pom.xml PRE-CREATION > trunk/hadooplibs/hadoop-test-0_24_0/pom.xml PRE-CREATION > trunk/hadooplibs/hadoop-test-1_0_0/pom.xml PRE-CREATION > trunk/hadooplibs/pom.xml PRE-CREATION > trunk/pom.xml 1239887 > trunk/sharelib/pom.xml 1239887 > trunk/sharelib/streaming/pom.xml 1239887 > trunk/src/main/assemblies/hadooplib.xml PRE-CREATION > trunk/src/main/assemblies/hadooplibs.xml PRE-CREATION > trunk/src/main/assemblies/partial-sharelib.xml 1239887 > trunk/webapp/pom.xml 1239887 > > Diff: https://reviews.apache.org/r/3726/diff > > > Testing > ------- > > Tested with 1.0.0 and 0.24.0-SNAPSHOTs (several testcases failures here but > that is still work in progress for the mr2 integration) > > > Thanks, > > Alejandro > >
