----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3726/#review4795 -----------------------------------------------------------
Ship it! +1 after addressing the doc comments for clarifications. trunk/docs/src/site/twiki/DG_QuickStart.twiki <https://reviews.apache.org/r/3726/#comment10557> It is not necessary to use the oozie-bundled hadoop libs. For example, H 0.24.2 is released and we are late or not packaging for some reason. There should be a way, for user to collect the necessary hadoop jars and put it into libext/. I know this is obvious. Just to mention in some way that user can bring there own hadoop library and copy it into libext. - Mohammad On 2012-02-03 05:44:33, Alejandro Abdelnur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3726/ > ----------------------------------------------------------- > > (Updated 2012-02-03 05:44:33) > > > 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 1240004 > trunk/core/src/test/java/org/apache/oozie/test/XTestCase.java 1240004 > trunk/docs/src/site/twiki/AG_Install.twiki 1240004 > trunk/docs/src/site/twiki/DG_QuickStart.twiki 1240004 > trunk/examples/pom.xml 1240004 > 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 1240004 > trunk/sharelib/pom.xml 1240004 > trunk/sharelib/streaming/pom.xml 1240004 > trunk/src/main/assemblies/hadooplib.xml PRE-CREATION > trunk/src/main/assemblies/hadooplibs.xml PRE-CREATION > trunk/src/main/assemblies/partial-sharelib.xml 1240004 > trunk/webapp/pom.xml 1240004 > > 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 > >
