> On None, Purshotam Shah wrote: > > core/src/main/java/org/apache/oozie/BundleActionBean.java, lines 53-55 > > <https://reviews.apache.org/r/20451/diff/2/?file=562213#file562213line53> > > > > This is not needed, we are not going to change bundle status to ignore. > > The ignore command is only for Coord not bundle job.
this is needed, bundle job need to remove bundle action in ignored for rerun, suspend, other operations > On None, Purshotam Shah wrote: > > core/src/main/java/org/apache/oozie/command/coord/CoordActionsIgnoreXCommand.java, > > line 66 > > <https://reviews.apache.org/r/20451/diff/2/?file=562221#file562221line66> > > > > it should be execute, i don't think children(WF) is involved here. use consistent naming convention with other TransitionXCommand > On None, Purshotam Shah wrote: > > core/src/main/java/org/apache/oozie/command/coord/CoordActionsIgnoreXCommand.java, > > line 104 > > <https://reviews.apache.org/r/20451/diff/2/?file=562221#file562221line104> > > > > We should set pending to coordob, so that StatusTransitService change > > the coord status. > > we don't want StatusTransitionService to pick up ignored coordinator job, since ignore status of job is agnostic to action status, so pending should be reset. - Ryota ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/20451/#review40799 ----------------------------------------------------------- On April 21, 2014, 5:32 p.m., Ryota Egashira wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/20451/ > ----------------------------------------------------------- > > (Updated April 21, 2014, 5:32 p.m.) > > > Review request for oozie. > > > Bugs: OOZIE-1791 > https://issues.apache.org/jira/browse/OOZIE-1791 > > > Repository: oozie-git > > > Description > ------- > > https://issues.apache.org/jira/browse/OOZIE-1791 > > WIP, just to show design, need to add test case and proper error handling, > etc.. > > > Diffs > ----- > > client/src/main/java/org/apache/oozie/cli/OozieCLI.java e3eb3b0 > client/src/main/java/org/apache/oozie/client/CoordinatorAction.java a0e42cd > client/src/main/java/org/apache/oozie/client/Job.java 8533165 > client/src/main/java/org/apache/oozie/client/OozieClient.java 40c9562 > client/src/main/java/org/apache/oozie/client/rest/RestConstants.java > a7fe06e > client/src/test/java/org/apache/oozie/client/rest/TestJsonToBean.java > 4091808 > core/src/main/java/org/apache/oozie/BundleActionBean.java 607f55b > core/src/main/java/org/apache/oozie/CoordinatorActionBean.java 03a7ed8 > core/src/main/java/org/apache/oozie/CoordinatorEngine.java 6a17ce4 > core/src/main/java/org/apache/oozie/ErrorCode.java f69d7a2 > core/src/main/java/org/apache/oozie/command/IgnoreTransitionXCommand.java > PRE-CREATION > > core/src/main/java/org/apache/oozie/command/bundle/BundleJobChangeXCommand.java > cd2c844 > > core/src/main/java/org/apache/oozie/command/bundle/BundleJobResumeXCommand.java > 1a11765 > > core/src/main/java/org/apache/oozie/command/bundle/BundleJobSuspendXCommand.java > 58afeac > core/src/main/java/org/apache/oozie/command/bundle/BundleJobXCommand.java > 2ddfbdb > core/src/main/java/org/apache/oozie/command/bundle/BundleKillXCommand.java > 3ab3273 > core/src/main/java/org/apache/oozie/command/bundle/BundleRerunXCommand.java > 90e205a > > core/src/main/java/org/apache/oozie/command/coord/CoordActionsIgnoreXCommand.java > PRE-CREATION > core/src/main/java/org/apache/oozie/command/coord/CoordChangeXCommand.java > fb31e9a > core/src/main/java/org/apache/oozie/command/coord/CoordKillXCommand.java > 2e5f6b4 > core/src/main/java/org/apache/oozie/command/coord/CoordSuspendXCommand.java > 0db2dad > core/src/main/java/org/apache/oozie/coord/CoordUtils.java 0319942 > > core/src/main/java/org/apache/oozie/executor/jpa/BundleActionQueryExecutor.java > 9c628ab > > core/src/main/java/org/apache/oozie/executor/jpa/CoordJobGetActionsStatusJPAExecutor.java > 2cee5f0 > core/src/main/java/org/apache/oozie/service/RecoveryService.java 2749bc4 > core/src/main/java/org/apache/oozie/service/StatusTransitService.java > e093c7d > core/src/main/java/org/apache/oozie/servlet/BaseJobServlet.java aa43e68 > core/src/main/java/org/apache/oozie/servlet/V0JobServlet.java 443ab6d > core/src/main/java/org/apache/oozie/servlet/V1JobServlet.java ac399e9 > core/src/main/java/org/apache/oozie/servlet/V2JobServlet.java 2d26599 > core/src/test/java/org/apache/oozie/action/hadoop/TestOozieJobInfo.java > 5aa9e6f > core/src/test/java/org/apache/oozie/client/TestOozieCLI.java 76c7a29 > > core/src/test/java/org/apache/oozie/command/bundle/TestBundleJobSuspendXCommand.java > 225d48e > > core/src/test/java/org/apache/oozie/command/bundle/TestBundleKillXCommand.java > 498ac0d > > core/src/test/java/org/apache/oozie/command/bundle/TestBundleStartXCommand.java > 7d6542a > > core/src/test/java/org/apache/oozie/command/coord/TestCoordActionsIgnoreXCommand.java > PRE-CREATION > > core/src/test/java/org/apache/oozie/command/coord/TestCoordChangeXCommand.java > 327ec90 > > core/src/test/java/org/apache/oozie/executor/jpa/TestBundleActionQueryExecutor.java > 3ae4d6c > > core/src/test/java/org/apache/oozie/servlet/MockCoordinatorEngineService.java > 9892d4b > core/src/test/java/org/apache/oozie/servlet/TestV2JobServlet.java aa5442c > docs/src/site/twiki/DG_CommandLineTool.twiki 5819b71 > docs/src/site/twiki/WebServicesAPI.twiki 351699d > > Diff: https://reviews.apache.org/r/20451/diff/ > > > Testing > ------- > > Not yet > > > Thanks, > > Ryota Egashira > >
