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

(Updated April 21, 2014, 9:29 p.m.)


Review request for oozie.


Changes
-------

-fixed some of remaining comments. 
-fixed bugs found in e2e test, now works fine.
-changed TestCoordActionsIgnoreXCommand
 


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 (updated)
-----

  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

Reply via email to