----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/5784/#review9114 -----------------------------------------------------------
trunk/core/src/main/java/org/apache/oozie/service/StatusTransitService.java <https://reviews.apache.org/r/5784/#comment19318> Now with the new states like RUNNINGWITHERROR, the status transit service need to check the job even if actions are pending. Hence we cant break out of this loop - Virag Kothari On July 10, 2012, 12:16 a.m., Virag Kothari wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/5784/ > ----------------------------------------------------------- > > (Updated July 10, 2012, 12:16 a.m.) > > > Review request for oozie. > > > Description > ------- > > 1) Added RUNNINGWITHERROR, PAUSEDWITHERROR and SUSPENDEDWITHERROR for > Coordinator and Bundle. > 2) Change in status of children will update the status of parent (e.g wf > rerun will update the state of coord) > > > This addresses bug OOZIE-708. > https://issues.apache.org/jira/browse/OOZIE-708 > > > Diffs > ----- > > trunk/core/src/main/java/org/apache/oozie/BundleActionBean.java 1359425 > trunk/core/src/main/java/org/apache/oozie/BundleJobBean.java 1359425 > trunk/core/src/main/java/org/apache/oozie/CoordinatorActionBean.java > 1359425 > trunk/core/src/main/java/org/apache/oozie/CoordinatorJobBean.java 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/RerunTransitionXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/ResumeTransitionXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/SuspendTransitionXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleJobChangeXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleJobResumeXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleJobSuspendXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleRerunXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/bundle/BundleStatusUpdateXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionInputCheckXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionReadyXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionUpdateXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordChangeXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordMaterializeTransitionXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordRerunXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordResumeXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/command/coord/CoordSuspendXCommand.java > 1359425 > trunk/core/src/main/java/org/apache/oozie/command/wf/ReRunXCommand.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/BundleActionsGetStatusPendingJPAExecutor.java > PRE-CREATION > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsStatusByPendingFalseJPAExecutor.java > 1359425 > > trunk/core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsStatusJPAExecutor.java > PRE-CREATION > trunk/core/src/main/java/org/apache/oozie/service/RecoveryService.java > 1359425 > trunk/core/src/main/java/org/apache/oozie/service/StatusTransitService.java > 1359425 > > trunk/core/src/test/java/org/apache/oozie/command/bundle/TestBundleJobSuspendXCommand.java > 1359425 > > trunk/core/src/test/java/org/apache/oozie/command/bundle/TestBundleRerunXCommand.java > 1359425 > > trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordChangeXCommand.java > 1359425 > > trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordRerunXCommand.java > 1359425 > > trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordResumeXCommand.java > 1359425 > > trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordSubmitXCommand.java > 1359425 > > trunk/core/src/test/java/org/apache/oozie/command/coord/TestCoordSuspendXCommand.java > 1359425 > > trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestBundleActionsStatusPendingJPAExecutor.java > PRE-CREATION > > trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsStatusByPendingFalseJPAExecutor.java > 1359425 > > trunk/core/src/test/java/org/apache/oozie/executor/jpa/TestCoordJobGetActionsStatusJPAExecutor.java > PRE-CREATION > > trunk/core/src/test/java/org/apache/oozie/service/TestStatusTransitService.java > 1359425 > trunk/docs/src/site/twiki/BundleFunctionalSpec.twiki 1359425 > trunk/docs/src/site/twiki/CoordinatorFunctionalSpec.twiki 1359425 > > Diff: https://reviews.apache.org/r/5784/diff/ > > > Testing > ------- > > Unit test cases added > QE testing in progress > > > Thanks, > > Virag Kothari > >
