Author: virag
Date: Wed May 15 17:53:22 2013
New Revision: 1482980

URL: http://svn.apache.org/r1482980
Log:
OOZIE-1356 Bundle job in PAUSEWITHERROR state does not goto SUSPENDEDWITHERROR 
state on suspending it (bowenzhangusa via virag)

Modified:
    
oozie/trunk/core/src/main/java/org/apache/oozie/command/SuspendTransitionXCommand.java
    
oozie/trunk/core/src/test/java/org/apache/oozie/command/bundle/TestBundleJobSuspendXCommand.java
    oozie/trunk/release-log.txt

Modified: 
oozie/trunk/core/src/main/java/org/apache/oozie/command/SuspendTransitionXCommand.java
URL: 
http://svn.apache.org/viewvc/oozie/trunk/core/src/main/java/org/apache/oozie/command/SuspendTransitionXCommand.java?rev=1482980&r1=1482979&r2=1482980&view=diff
==============================================================================
--- 
oozie/trunk/core/src/main/java/org/apache/oozie/command/SuspendTransitionXCommand.java
 (original)
+++ 
oozie/trunk/core/src/main/java/org/apache/oozie/command/SuspendTransitionXCommand.java
 Wed May 15 17:53:22 2013
@@ -52,7 +52,7 @@ public abstract class SuspendTransitionX
         else if (job.getStatus() == Job.Status.RUNNING) {
             job.setStatus(Job.Status.SUSPENDED);
         }
-        else if (job.getStatus() == Job.Status.RUNNINGWITHERROR){
+        else if (job.getStatus() == Job.Status.RUNNINGWITHERROR || 
job.getStatus() == Job.Status.PAUSEDWITHERROR){
             job.setStatus(Job.Status.SUSPENDEDWITHERROR);
         }
         else if (job.getStatus() == Job.Status.PAUSED) {

Modified: 
oozie/trunk/core/src/test/java/org/apache/oozie/command/bundle/TestBundleJobSuspendXCommand.java
URL: 
http://svn.apache.org/viewvc/oozie/trunk/core/src/test/java/org/apache/oozie/command/bundle/TestBundleJobSuspendXCommand.java?rev=1482980&r1=1482979&r2=1482980&view=diff
==============================================================================
--- 
oozie/trunk/core/src/test/java/org/apache/oozie/command/bundle/TestBundleJobSuspendXCommand.java
 (original)
+++ 
oozie/trunk/core/src/test/java/org/apache/oozie/command/bundle/TestBundleJobSuspendXCommand.java
 Wed May 15 17:53:22 2013
@@ -95,6 +95,27 @@ public class TestBundleJobSuspendXComman
         job = jpaService.execute(bundleJobGetCmd);
         assertEquals(Job.Status.SUSPENDEDWITHERROR, job.getStatus());
     }
+
+    /**
+     * Test : Suspend bundle job in PAUSEDWITHERROR state
+     *
+     * @throws Exception
+     */
+    public void testBundleSuspendWithError2() throws Exception {
+        BundleJobBean job = 
this.addRecordToBundleJobTable(Job.Status.PAUSEDWITHERROR, false);
+
+        JPAService jpaService = Services.get().get(JPAService.class);
+        assertNotNull(jpaService);
+        BundleJobGetJPAExecutor bundleJobGetCmd = new 
BundleJobGetJPAExecutor(job.getId());
+        job = jpaService.execute(bundleJobGetCmd);
+        assertEquals(Job.Status.PAUSEDWITHERROR, job.getStatus());
+
+        new BundleJobSuspendXCommand(job.getId()).call();
+
+        job = jpaService.execute(bundleJobGetCmd);
+        assertEquals(Job.Status.SUSPENDEDWITHERROR, job.getStatus());
+    }
+
     /**
      * Test : Suspend bundle job
      *

Modified: oozie/trunk/release-log.txt
URL: 
http://svn.apache.org/viewvc/oozie/trunk/release-log.txt?rev=1482980&r1=1482979&r2=1482980&view=diff
==============================================================================
--- oozie/trunk/release-log.txt (original)
+++ oozie/trunk/release-log.txt Wed May 15 17:53:22 2013
@@ -1,5 +1,6 @@
 -- Oozie 4.1.0 release (trunk - unreleased)
 
+OOZIE-1356 Bundle job in PAUSEWITHERROR state does not goto SUSPENDEDWITHERROR 
state on suspending it (bowenzhangusa via virag)
 OOZIE-1313 coverage fix for org.apache.oozie.client (aleksgor via virag)
 OOZIE-1360 Oozie CLI shows created time of workflow as started time (ryota via 
virag)
 OOZIE-1359 mention default value of throttle in doc (ryota via virag)


Reply via email to