Author: virag
Date: Fri Jul 26 23:34:34 2013
New Revision: 1507517

URL: http://svn.apache.org/r1507517
Log:
OOZIE-1469 loadState() and/or verifyPrecondition() are blank in some commands 
(virag)

Modified:
    
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/coord/CoordActionUpdateXCommand.java
    
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/ActionCheckXCommand.java
    
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/CompletedActionXCommand.java
    
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/ReRunXCommand.java
    
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/SuspendXCommand.java
    oozie/branches/branch-4.0/release-log.txt

Modified: 
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/coord/CoordActionUpdateXCommand.java
URL: 
http://svn.apache.org/viewvc/oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/coord/CoordActionUpdateXCommand.java?rev=1507517&r1=1507516&r2=1507517&view=diff
==============================================================================
--- 
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/coord/CoordActionUpdateXCommand.java
 (original)
+++ 
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/coord/CoordActionUpdateXCommand.java
 Fri Jul 26 23:34:34 2013
@@ -215,6 +215,7 @@ public class CoordActionUpdateXCommand e
      */
     @Override
     protected void loadState() throws CommandException {
+        eagerLoadState();
     }
 
     /* (non-Javadoc)

Modified: 
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/ActionCheckXCommand.java
URL: 
http://svn.apache.org/viewvc/oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/ActionCheckXCommand.java?rev=1507517&r1=1507516&r2=1507517&view=diff
==============================================================================
--- 
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/ActionCheckXCommand.java
 (original)
+++ 
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/ActionCheckXCommand.java
 Fri Jul 26 23:34:34 2013
@@ -134,6 +134,7 @@ public class ActionCheckXCommand extends
 
     @Override
     protected void loadState() throws CommandException {
+        eagerLoadState();
     }
 
     @Override

Modified: 
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/CompletedActionXCommand.java
URL: 
http://svn.apache.org/viewvc/oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/CompletedActionXCommand.java?rev=1507517&r1=1507516&r2=1507517&view=diff
==============================================================================
--- 
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/CompletedActionXCommand.java
 (original)
+++ 
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/CompletedActionXCommand.java
 Fri Jul 26 23:34:34 2013
@@ -6,9 +6,9 @@
  * to you under the Apache License, Version 2.0 (the
  * "License"); you may not use this file except in compliance
  * with the License.  You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -52,7 +52,7 @@ public class CompletedActionXCommand ext
 
     /*
      * (non-Javadoc)
-     * 
+     *
      * @see org.apache.oozie.command.XCommand#eagerLoadState()
      */
     @Override
@@ -75,7 +75,7 @@ public class CompletedActionXCommand ext
 
     /*
      * (non-Javadoc)
-     * 
+     *
      * @see org.apache.oozie.command.XCommand#eagerVerifyPrecondition()
      */
     @Override
@@ -88,7 +88,7 @@ public class CompletedActionXCommand ext
 
     /*
      * (non-Javadoc)
-     * 
+     *
      * @see org.apache.oozie.command.XCommand#execute()
      */
     @Override
@@ -114,7 +114,7 @@ public class CompletedActionXCommand ext
 
     /*
      * (non-Javadoc)
-     * 
+     *
      * @see org.apache.oozie.command.XCommand#isLockRequired()
      */
     @Override
@@ -124,19 +124,21 @@ public class CompletedActionXCommand ext
 
     /*
      * (non-Javadoc)
-     * 
+     *
      * @see org.apache.oozie.command.XCommand#loadState()
      */
     @Override
     protected void loadState() throws CommandException {
+        eagerLoadState();
     }
 
     /*
      * (non-Javadoc)
-     * 
+     *
      * @see org.apache.oozie.command.XCommand#verifyPrecondition()
      */
     @Override
     protected void verifyPrecondition() throws CommandException, 
PreconditionException {
+        eagerVerifyPrecondition();
     }
 }

Modified: 
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/ReRunXCommand.java
URL: 
http://svn.apache.org/viewvc/oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/ReRunXCommand.java?rev=1507517&r1=1507516&r2=1507517&view=diff
==============================================================================
--- 
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/ReRunXCommand.java
 (original)
+++ 
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/ReRunXCommand.java
 Fri Jul 26 23:34:34 2013
@@ -392,6 +392,7 @@ public class ReRunXCommand extends Workf
      */
     @Override
     protected void loadState() throws CommandException {
+        eagerLoadState();
     }
 
     /* (non-Javadoc)
@@ -399,5 +400,6 @@ public class ReRunXCommand extends Workf
      */
     @Override
     protected void verifyPrecondition() throws CommandException, 
PreconditionException {
+        eagerVerifyPrecondition();
     }
 }

Modified: 
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/SuspendXCommand.java
URL: 
http://svn.apache.org/viewvc/oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/SuspendXCommand.java?rev=1507517&r1=1507516&r2=1507517&view=diff
==============================================================================
--- 
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/SuspendXCommand.java
 (original)
+++ 
oozie/branches/branch-4.0/core/src/main/java/org/apache/oozie/command/wf/SuspendXCommand.java
 Fri Jul 26 23:34:34 2013
@@ -195,7 +195,7 @@ public class SuspendXCommand extends Wor
      */
     @Override
     protected void loadState() throws CommandException {
-
+        eagerLoadState();
     }
 
     /* (non-Javadoc)
@@ -203,5 +203,6 @@ public class SuspendXCommand extends Wor
      */
     @Override
     protected void verifyPrecondition() throws CommandException, 
PreconditionException {
+        eagerVerifyPrecondition();
     }
 }

Modified: oozie/branches/branch-4.0/release-log.txt
URL: 
http://svn.apache.org/viewvc/oozie/branches/branch-4.0/release-log.txt?rev=1507517&r1=1507516&r2=1507517&view=diff
==============================================================================
--- oozie/branches/branch-4.0/release-log.txt (original)
+++ oozie/branches/branch-4.0/release-log.txt Fri Jul 26 23:34:34 2013
@@ -1,5 +1,6 @@
 -- Oozie 4.0.0 release
 
+OOZIE-1469 loadState() and/or verifyPrecondition() are blank in some commands 
(virag)
 OOZIE-1467 Bundle not killed if coordinator fails due to db exception 
(rohini,virag via virag)
 OOZIE-1466 current EL should not check for less than or equal to zero (rohini)
 OOZIE-1465 Making constants in CoordELFunctions public for el extensions 
(shwethags via rohini)


Reply via email to