----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/3047/#review3805 -----------------------------------------------------------
trunk/core/src/main/java/org/apache/oozie/DagELFunctions.java <https://reviews.apache.org/r/3047/#comment8620> MapReduceActionExecutor already has this. Could you merge to one place? trunk/core/src/main/java/org/apache/oozie/DagELFunctions.java <https://reviews.apache.org/r/3047/#comment8621> it can be private trunk/core/src/main/java/org/apache/oozie/action/hadoop/ActionType.java <https://reviews.apache.org/r/3047/#comment8625> Why this enum has its own class? Can it be in any existed class? It can be part of ActionStats. trunk/core/src/main/java/org/apache/oozie/action/hadoop/MapReduceActionExecutor.java <https://reviews.apache.org/r/3047/#comment8628> private XLog log = XLog.getLog(getClass()); This already defined in the beginning of class. - Angelo K. On 2011-12-09 22:26:29, params wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/3047/ > ----------------------------------------------------------- > > (Updated 2011-12-09 22:26:29) > > > Review request for oozie, Mohammad Islam and Angelo K. Huang. > > > Summary > ------- > > OOZIE-588: Oozie to allow drill down to hadoop job's details > > Summary: > https://issues.apache.org/jira/browse/OOZIE-588 > > Features Implemented: > - Expose action stats information (hadoop counters and pig stats) through CLI > -info -verbose option > - Expose external child IDs (eg: hadoop job IDs for the pig action) through > CLI -info -verbose option > - Expose job counter info as EL Functions > - Expose external child IDs as EL Functions > - User level configuration to specify if stats should be collected or not > (with a default setting) > - Oozie level configuration to specify the max allowable size for stats > > > This addresses bug OOZIE-588. > https://issues.apache.org/jira/browse/OOZIE-588 > > > Diffs > ----- > > trunk/client/src/main/java/org/apache/oozie/cli/OozieCLI.java 1212662 > trunk/client/src/main/java/org/apache/oozie/client/WorkflowAction.java > 1212662 > trunk/client/src/main/java/org/apache/oozie/client/rest/JsonTags.java > 1212662 > trunk/client/src/main/java/org/apache/oozie/client/rest/JsonToBean.java > 1212662 > trunk/core/src/main/java/org/apache/oozie/DagELFunctions.java 1212662 > trunk/core/src/main/java/org/apache/oozie/WorkflowActionBean.java 1212662 > trunk/core/src/main/java/org/apache/oozie/action/ActionExecutor.java > 1212662 > trunk/core/src/main/java/org/apache/oozie/action/hadoop/ActionStats.java > PRE-CREATION > trunk/core/src/main/java/org/apache/oozie/action/hadoop/ActionType.java > PRE-CREATION > trunk/core/src/main/java/org/apache/oozie/action/hadoop/MRStats.java > PRE-CREATION > > trunk/core/src/main/java/org/apache/oozie/action/hadoop/MapReduceActionExecutor.java > 1212662 > > trunk/core/src/main/java/org/apache/oozie/client/rest/JsonWorkflowAction.java > 1212662 > trunk/core/src/main/java/org/apache/oozie/command/wf/ActionCommand.java > 1212662 > trunk/core/src/main/java/org/apache/oozie/command/wf/ActionXCommand.java > 1212662 > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/WorkflowActionGetJPAExecutor.java > 1212662 > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/WorkflowActionSubsetGetJPAExecutor.java > 1212662 > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/WorkflowActionsGetForJobJPAExecutor.java > 1212662 > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/WorkflowActionsRunningGetJPAExecutor.java > 1212662 > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/WorkflowJobGetActionsJPAExecutor.java > 1212662 > trunk/core/src/main/java/org/apache/oozie/store/WorkflowStore.java 1212662 > trunk/core/src/test/java/org/apache/oozie/TestActionBean.java 1212662 > > trunk/core/src/test/java/org/apache/oozie/action/decision/TestDecisionActionExecutor.java > 1212662 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/ActionExecutorTestCase.java > 1212662 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestHadoopELFunctions.java > 1212662 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceActionError.java > 1212662 > > trunk/core/src/test/java/org/apache/oozie/action/hadoop/TestMapReduceActionExecutor.java > 1212662 > > trunk/core/src/test/java/org/apache/oozie/action/ssh/TestSshActionExecutor.java > 1212662 > > trunk/core/src/test/java/org/apache/oozie/client/rest/TestJsonWorkflowAction.java > 1212662 > > Diff: https://reviews.apache.org/r/3047/diff > > > Testing > ------- > > - Tested against existing test cases and added tests where required. > - Tests for the new classes MRStats and OoziePigStats have not been added. > This is work in progress. > > > Thanks, > > params > >
