----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15225/#review29157 -----------------------------------------------------------
trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionStartXCommand.java <https://reviews.apache.org/r/15225/#comment56298> this is to breakdown wf submit and start. Otherwise, wf goes ahead, starts, and invokes updateParentIfNecessary based on parentId - However, coord action's externalId pointing to his wf has not been persisted yet - causing NPE. - Mona Chitnis On Nov. 20, 2013, 2:31 a.m., Mona Chitnis wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15225/ > ----------------------------------------------------------- > > (Updated Nov. 20, 2013, 2:31 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/DagEngine.java 1543124 > trunk/core/src/main/java/org/apache/oozie/command/XCommand.java 1543124 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionInputCheckXCommand.java > 1543124 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionReadyXCommand.java > 1543124 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionStartXCommand.java > 1543124 > > trunk/core/src/main/java/org/apache/oozie/command/wf/ActionCheckXCommand.java > 1543124 > trunk/core/src/main/java/org/apache/oozie/command/wf/ActionEndXCommand.java > 1543124 > > trunk/core/src/main/java/org/apache/oozie/command/wf/ActionStartXCommand.java > 1543124 > trunk/core/src/main/java/org/apache/oozie/command/wf/SignalXCommand.java > 1543124 > trunk/core/src/main/java/org/apache/oozie/service/WorkflowStoreService.java > 1543124 > > trunk/core/src/test/java/org/apache/oozie/TestCoordinatorEngineStreamLog.java > 1543124 > trunk/core/src/test/java/org/apache/oozie/action/TestActionFailover.java > 1543124 > > trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordActionInputCheckXCommand.java > 1543124 > > trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordActionNotificationXCommand.java > 1543124 > > trunk/core/src/test/java/org/apache/oozie/command/wf/TestNotificationXCommand.java > 1543124 > > trunk/core/src/test/java/org/apache/oozie/command/wf/TestSignalXCommand.java > 1543124 > trunk/core/src/test/java/org/apache/oozie/event/TestEventGeneration.java > 1543124 > trunk/core/src/test/java/org/apache/oozie/service/TestRecoveryService.java > 1543124 > trunk/core/src/test/java/org/apache/oozie/sla/TestSLAEventGeneration.java > 1543124 > trunk/core/src/test/java/org/apache/oozie/test/XDataTestCase.java 1543124 > trunk/core/src/test/resources/wf-fork.xml 1543124 > > Diff: https://reviews.apache.org/r/15225/diff/ > > > Testing > ------- > > unit tests fixed. stress testing performed to validate the fast start case. > > > Thanks, > > Mona Chitnis > >
