----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15225/#review29084 -----------------------------------------------------------
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionReadyXCommand.java <https://reviews.apache.org/r/15225/#comment56204> how will locking work here? Is it actually getting queued again? trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionUpdateXCommand.java <https://reviews.apache.org/r/15225/#comment56205> not required trunk/core/src/main/java/org/apache/oozie/command/wf/ActionEndXCommand.java <https://reviews.apache.org/r/15225/#comment56206> trunk/core/src/main/java/org/apache/oozie/command/wf/ActionEndXCommand.java <https://reviews.apache.org/r/15225/#comment56211> Remove the variable and do check for control node directly trunk/core/src/main/java/org/apache/oozie/command/wf/ActionStartXCommand.java <https://reviews.apache.org/r/15225/#comment56212> controlNode && wfAction.isExecutionComplete() && context.isExecuted() trunk/core/src/main/java/org/apache/oozie/executor/jpa/BatchQueryExecutor.java <https://reviews.apache.org/r/15225/#comment56217> Not required. it will already throw JPAExecutorException trunk/core/src/test/java/org/apache/oozie/action/TestActionFailover.java <https://reviews.apache.org/r/15225/#comment56213> Why does this change happen? trunk/core/src/test/java/org/apache/oozie/command/wf/TestSignalXCommand.java <https://reviews.apache.org/r/15225/#comment56214> Can you explain this change? trunk/core/src/test/java/org/apache/oozie/event/TestEventGeneration.java <https://reviews.apache.org/r/15225/#comment56216> Can you explain this change? trunk/core/src/test/java/org/apache/oozie/test/XDataTestCase.java <https://reviews.apache.org/r/15225/#comment56215> Can you explain this change? - Rohini Palaniswamy On Nov. 9, 2013, 4:20 a.m., Mona Chitnis wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15225/ > ----------------------------------------------------------- > > (Updated Nov. 9, 2013, 4:20 a.m.) > > > Review request for oozie. > > > Bugs: OOZIE-1581 > https://issues.apache.org/jira/browse/OOZIE-1581 > > > Repository: oozie > > > Description > ------- > > Patch for review updated from the attachments on JIRA. > 1. Included change to call coordinator action Start synchronously from Ready > command. > 2. Made Decision Action Executor extend from Control Action Executor. That > way the synchronous commands can be executed for all such 'connector' actions. > 3. Fixed failing unit tests. > 4. TestSLAEventGeneration unit test is failing - WIP. > > > Diffs > ----- > > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionReadyXCommand.java > 1538746 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionUpdateXCommand.java > 1538746 > trunk/core/src/main/java/org/apache/oozie/command/wf/ActionEndXCommand.java > 1538746 > > trunk/core/src/main/java/org/apache/oozie/command/wf/ActionStartXCommand.java > 1538746 > trunk/core/src/main/java/org/apache/oozie/command/wf/SignalXCommand.java > 1538746 > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/BatchQueryExecutor.java > 1538746 > trunk/core/src/main/java/org/apache/oozie/service/WorkflowStoreService.java > 1538746 > > trunk/core/src/test/java/org/apache/oozie/TestCoordinatorEngineStreamLog.java > 1538746 > trunk/core/src/test/java/org/apache/oozie/action/TestActionFailover.java > 1538746 > > trunk/core/src/test/java/org/apache/oozie/command/wf/TestSignalXCommand.java > 1538746 > trunk/core/src/test/java/org/apache/oozie/event/TestEventGeneration.java > 1538746 > trunk/core/src/test/java/org/apache/oozie/service/TestRecoveryService.java > 1538746 > trunk/core/src/test/java/org/apache/oozie/sla/TestSLAEventGeneration.java > 1538746 > trunk/core/src/test/java/org/apache/oozie/test/XDataTestCase.java 1538746 > trunk/core/src/test/resources/wf-fork.xml 1538746 > > Diff: https://reviews.apache.org/r/15225/diff/ > > > Testing > ------- > > unit tests fixed. stress testing performed to validate the fast start case. > > > Thanks, > > Mona Chitnis > >
