> On Sept. 4, 2014, 8:55 p.m., Mona Chitnis wrote:
> > core/src/main/java/org/apache/oozie/command/bundle/BundleStatusTransitXCommand.java,
> > line 177
> > <https://reviews.apache.org/r/24948/diff/1/?file=668667#file668667line177>
> >
> > related question, is this situation possible? - job status is PAUSED ||
> > PWE, and bundle action status is RWE?
>
> Purshotam Shah wrote:
> Yes, BundlePauseXCommand only set bundle status to pause. Bundle status
> can still be in running state.
Ok. I just checked that BundlePauseXCommand and CoordPauseXCommand have empty
implementations of pauseChildren()
@Override
public void pauseChildren() throws CommandException {
// TODO - need revisit when revisiting coord job status redesign;
}
- Mona
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/24948/#review52346
-----------------------------------------------------------
On Sept. 8, 2014, 9:15 p.m., Purshotam Shah wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/24948/
> -----------------------------------------------------------
>
> (Updated Sept. 8, 2014, 9:15 p.m.)
>
>
> Review request for oozie.
>
>
> Bugs: OOZIE-1940
> https://issues.apache.org/jira/browse/OOZIE-1940
>
>
> Repository: oozie-git
>
>
> Description
> -------
>
> StatusTransitService has race condition
>
>
> Diffs
> -----
>
> core/src/main/java/org/apache/oozie/BundleActionBean.java 5d85a4d
> core/src/main/java/org/apache/oozie/BundleJobBean.java 0f1670a
> core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 795bf63
> core/src/main/java/org/apache/oozie/CoordinatorJobBean.java 8fd53f1
> core/src/main/java/org/apache/oozie/ErrorCode.java 88a2c67
> core/src/main/java/org/apache/oozie/command/StatusTransitXCommand.java
> e69de29
>
> core/src/main/java/org/apache/oozie/command/bundle/BundleStatusTransitXCommand.java
> e69de29
>
> core/src/main/java/org/apache/oozie/command/coord/CoordStatusTransitXCommand.java
> e69de29
>
> core/src/main/java/org/apache/oozie/executor/jpa/BundleJobQueryExecutor.java
> 36cd968
>
> core/src/main/java/org/apache/oozie/executor/jpa/CoordActionQueryExecutor.java
> 3008393
> core/src/main/java/org/apache/oozie/executor/jpa/CoordJobQueryExecutor.java
> 04e6e29
> core/src/main/java/org/apache/oozie/service/StatusTransitService.java
> 21ac25f
> core/src/test/java/org/apache/oozie/service/TestStatusTransitService.java
> bb99138
>
> Diff: https://reviews.apache.org/r/24948/diff/
>
>
> Testing
> -------
>
> UTC
>
>
> Thanks,
>
> Purshotam Shah
>
>
