> On 2012-02-22 03:21:41, Mohammad Islam wrote: > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/HiveMain.java, line > > 107 > > <https://reviews.apache.org/r/3991/diff/4/?file=78576#file78576line107> > > > > space
will fix > On 2012-02-22 03:21:41, Mohammad Islam wrote: > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/SharelibUtils.java, > > line 58 > > <https://reviews.apache.org/r/3991/diff/4/?file=78580#file78580line58> > > > > typo finClasspathFile ==> finClassPathFile the typo is 'fin', it should be 'findClasspathFile', classpath is treated as a single word. > On 2012-02-22 03:21:41, Mohammad Islam wrote: > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/SharelibUtils.java, > > line 75 > > <https://reviews.apache.org/r/3991/diff/4/?file=78580#file78580line75> > > > > typo classpathFile -->classPathFile classpath is normally treated as a single word > On 2012-02-22 03:21:41, Mohammad Islam wrote: > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/SharelibUtils.java, > > line 77 > > <https://reviews.apache.org/r/3991/diff/4/?file=78580#file78580line77> > > > > is it possible that line could be 'null' that is file is empty? no, if the file is empy you get an emtpy string. > On 2012-02-22 03:21:41, Mohammad Islam wrote: > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestHiveActionExecutor.java, > > line 238 > > <https://reviews.apache.org/r/3991/diff/4/?file=78581#file78581line238> > > > > does it needs the framework property? > > conf.set("mapreduce.framework.name", "yarn"); yes, 0.23 need this, and pre 0.23 ignores it. we already have this in other testcases and in the KerberosHadoopAccessorService > On 2012-02-22 03:21:41, Mohammad Islam wrote: > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestHiveMain.java, > > line 147 > > <https://reviews.apache.org/r/3991/diff/4/?file=78582#file78582line147> > > > > need this? conf.set("mapreduce.framework.name", "yarn"); same as prev comment > On 2012-02-22 03:21:41, Mohammad Islam wrote: > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestSqoopActionExecutor.java, > > line 317 > > <https://reviews.apache.org/r/3991/diff/4/?file=78585#file78585line317> > > > > space ok > On 2012-02-22 03:21:41, Mohammad Islam wrote: > > trunk/pom.xml, line 440 > > <https://reviews.apache.org/r/3991/diff/4/?file=78595#file78595line440> > > > > space ok - Alejandro ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3991/#review5262 ----------------------------------------------------------- On 2012-02-21 23:16:56, Alejandro Abdelnur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3991/ > ----------------------------------------------------------- > > (Updated 2012-02-21 23:16:56) > > > Review request for oozie. > > > Summary > ------- > > OOZIE-700,OOZIE-702,OOZIE-703,OOZIE-704 combo review. > > These patches build on top of each other, the first patch is the first > revision, the second patch is second revision, etc, etc. Review them one by > one by selecting a diff between the current and previous revision of the patch > > > This addresses bugs OOZIE-700, OOZIE-702, OOZIE-703, and OOZIE-704. > https://issues.apache.org/jira/browse/OOZIE-700 > https://issues.apache.org/jira/browse/OOZIE-702 > https://issues.apache.org/jira/browse/OOZIE-703 > https://issues.apache.org/jira/browse/OOZIE-704 > > > Diffs > ----- > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/HiveMain.java > 1292066 > trunk/core/src/main/java/org/apache/oozie/action/hadoop/LauncherMain.java > 1292066 > trunk/core/src/main/java/org/apache/oozie/action/hadoop/PigMain.java > 1292066 > trunk/core/src/main/java/org/apache/oozie/action/hadoop/SqoopMain.java > 1292066 > trunk/core/src/test/java/org/apache/oozie/action/hadoop/SharelibUtils.java > PRE-CREATION > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestHiveActionExecutor.java > 1292066 > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestHiveMain.java > 1292066 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestPigActionExecutor.java > 1292066 > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestPigMain.java > 1292066 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestSqoopActionExecutor.java > 1292066 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestStreamingMain.java > 1292066 > trunk/core/src/test/java/org/apache/oozie/test/XTestCase.java 1292066 > trunk/hadooplibs/hadoop-0_23_1/pom.xml 1292066 > trunk/hadooplibs/hadoop-0_23_2/pom.xml PRE-CREATION > trunk/hadooplibs/hadoop-1_0_1/pom.xml PRE-CREATION > trunk/hadooplibs/hadoop-test-0_23_1/pom.xml 1292066 > trunk/hadooplibs/hadoop-test-0_23_2/pom.xml PRE-CREATION > trunk/hadooplibs/hadoop-test-1_0_1/pom.xml PRE-CREATION > trunk/hadooplibs/pom.xml 1292066 > trunk/pom.xml 1292066 > trunk/sharelib/hive/pom.xml 1292066 > trunk/sharelib/oozie/pom.xml 1292066 > trunk/sharelib/pig/pom.xml 1292066 > trunk/sharelib/sqoop/pom.xml 1292066 > trunk/sharelib/streaming/pom.xml 1292066 > > Diff: https://reviews.apache.org/r/3991/diff > > > Testing > ------- > > tested with hadoop-1 and hadoop-0.23. > > There are still some failures but getting closer. > > One big issue is Pig JAR which does not run with Hadoop 0.23 (PIG-2546). I've > tested by installing in my local m2 cache a Pig JAR compiled with 0.23 > > Note, for hive action to work, Hadoop 0.23 must be patched with MAPREDUCE-3884 > > > Thanks, > > Alejandro > >
