You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@oozie.apache.org by Mona Chitnis <ch...@yahoo-inc.com> on 2012/07/12 20:59:29 UTC

Re: Review Request: OOZIE-708: Update the parent entity status dynamically

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/5784/#review9113
-----------------------------------------------------------



trunk/core/src/main/java/org/apache/oozie/service/StatusTransitService.java
<https://reviews.apache.org/r/5784/#comment19316>

    why is break removed?



trunk/core/src/main/java/org/apache/oozie/service/StatusTransitService.java
<https://reviews.apache.org/r/5784/#comment19317>

    good flag. helps to reduce code later on


- Mona Chitnis


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
> 
>