----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/4236/#review5699 -----------------------------------------------------------
trunk/core/src/main/java/org/apache/oozie/action/hadoop/FsActionExecutor.java <https://reviews.apache.org/r/4236/#comment12409> storing the HadoopAccessorService in a temp variable (such 'has') used it here. Same thing is done in previous method at line 139 trunk/core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java <https://reviews.apache.org/r/4236/#comment12410> namenode needs to be added into jobconf? trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleSubmitXCommand.java <https://reviews.apache.org/r/4236/#comment12411> These tree lines of code is being used many places. Can't we make it a new utility method and call it from every places? trunk/core/src/main/java/org/apache/oozie/command/wf/WfEndXCommand.java <https://reviews.apache.org/r/4236/#comment12412> same here. create a utility method and call it. - Mohammad On 2012-03-08 00:29:07, Alejandro Abdelnur wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/4236/ > ----------------------------------------------------------- > > (Updated 2012-03-08 00:29:07) > > > Review request for oozie. > > > Summary > ------- > > First patch is OOZIE-744 > Second patch is OOZIE-744 + OOZIE-745 > Third patch is OOZIE-744 + OOZIE-745 + OOZIE-746 > > > This addresses bugs OOZIE-744, OOZIE-745 and OOZIE-746. > https://issues.apache.org/jira/browse/OOZIE-744 > https://issues.apache.org/jira/browse/OOZIE-745 > https://issues.apache.org/jira/browse/OOZIE-746 > > > Diffs > ----- > > trunk/core/src/main/conf/hadoop-config.xml 1298218 > trunk/core/src/main/java/org/apache/oozie/ErrorCode.java 1298218 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/FsActionExecutor.java > 1298218 > trunk/core/src/main/java/org/apache/oozie/action/hadoop/FsELFunctions.java > 1298218 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/JavaActionExecutor.java > 1298218 > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/MapReduceActionExecutor.java > 1298218 > > trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleSubmitXCommand.java > 1298218 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionInputCheckXCommand.java > 1298218 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordSubmitXCommand.java > 1298218 > trunk/core/src/main/java/org/apache/oozie/command/wf/ActionXCommand.java > 1298218 > trunk/core/src/main/java/org/apache/oozie/command/wf/ReRunXCommand.java > 1298218 > trunk/core/src/main/java/org/apache/oozie/command/wf/SubmitXCommand.java > 1298218 > trunk/core/src/main/java/org/apache/oozie/command/wf/WfEndXCommand.java > 1298218 > trunk/core/src/main/java/org/apache/oozie/coord/CoordELFunctions.java > 1298218 > trunk/core/src/main/java/org/apache/oozie/service/AuthorizationService.java > 1298218 > > trunk/core/src/main/java/org/apache/oozie/service/HadoopAccessorService.java > 1298218 > trunk/core/src/main/java/org/apache/oozie/service/WorkflowAppService.java > 1298218 > trunk/core/src/main/java/org/apache/oozie/util/JobUtils.java 1298218 > trunk/core/src/test/java/org/apache/oozie/action/TestActionFailover.java > 1298218 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestDistCpActionExecutor.java > 1298218 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestHiveActionExecutor.java > 1298218 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestJavaActionExecutor.java > 1298218 > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestLauncher.java > 1298218 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceActionError.java > 1298218 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceActionExecutor.java > 1298218 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestPigActionExecutor.java > 1298218 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestShellActionExecutor.java > 1298218 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestSqoopActionExecutor.java > 1298218 > > trunk/core/src/test/java/org/apache/oozie/command/wf/TestActionCheckXCommand.java > 1298218 > > trunk/core/src/test/java/org/apache/oozie/command/wf/TestActionStartXCommand.java > 1298218 > > trunk/core/src/test/java/org/apache/oozie/service/TestAuthorizationService.java > 1298218 > > trunk/core/src/test/java/org/apache/oozie/service/TestHadoopAccessorService.java > 1298218 > trunk/core/src/test/java/org/apache/oozie/service/TestRecoveryService.java > 1298218 > trunk/core/src/test/java/org/apache/oozie/test/XFsTestCase.java 1298218 > > trunk/examples/src/test/java/org/apache/oozie/example/TestLocalOozieExample.java > 1298218 > > Diff: https://reviews.apache.org/r/4236/diff > > > Testing > ------- > > run all tests against Hadoop 1 > > built deployed Oozie and submitted a MR WF against Hadoop 1 and Hadoop 0.23 > > > Thanks, > > Alejandro > >
