Author: rohini
Date: Fri Jul 12 20:41:08 2013
New Revision: 1502680

URL: http://svn.apache.org/r1502680
Log:
OOZIE-1451 CoordActionInputCheckX does a redundant eagerLoadState (rohini)

Modified:
    
oozie/trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionInputCheckXCommand.java
    oozie/trunk/release-log.txt

Modified: 
oozie/trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionInputCheckXCommand.java
URL: 
http://svn.apache.org/viewvc/oozie/trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionInputCheckXCommand.java?rev=1502680&r1=1502679&r2=1502680&view=diff
==============================================================================
--- 
oozie/trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionInputCheckXCommand.java
 (original)
+++ 
oozie/trunk/core/src/main/java/org/apache/oozie/command/coord/CoordActionInputCheckXCommand.java
 Fri Jul 12 20:41:08 2013
@@ -87,9 +87,6 @@ public class CoordActionInputCheckXComma
         this.jobId = jobId;
     }
 
-    /* (non-Javadoc)
-     * @see org.apache.oozie.command.XCommand#execute()
-     */
     @Override
     protected Void execute() throws CommandException {
         LOG.info("[" + actionId + "]::ActionInputCheck:: Action is in WAITING 
state.");
@@ -553,34 +550,16 @@ public class CoordActionInputCheckXComma
         return null;
     }
 
-    /* (non-Javadoc)
-     * @see org.apache.oozie.command.XCommand#getEntityKey()
-     */
     @Override
     public String getEntityKey() {
         return this.jobId;
     }
 
-    /* (non-Javadoc)
-     * @see org.apache.oozie.command.XCommand#isLockRequired()
-     */
     @Override
     protected boolean isLockRequired() {
         return true;
     }
 
-    /* (non-Javadoc)
-     * @see org.apache.oozie.command.XCommand#eagerLoadState()
-     */
-    // TODO - why loadState() is being called from eagerLoadState();
-    @Override
-    protected void eagerLoadState() throws CommandException {
-        loadState();
-    }
-
-    /* (non-Javadoc)
-     * @see org.apache.oozie.command.XCommand#loadState()
-     */
     @Override
     protected void loadState() throws CommandException {
         if (jpaService == null) {
@@ -596,9 +575,6 @@ public class CoordActionInputCheckXComma
         LogUtils.setLogInfo(coordAction, logInfo);
     }
 
-    /* (non-Javadoc)
-     * @see org.apache.oozie.command.XCommand#verifyPrecondition()
-     */
     @Override
     protected void verifyPrecondition() throws CommandException, 
PreconditionException {
         if (coordAction.getStatus() != CoordinatorActionBean.Status.WAITING) {
@@ -621,9 +597,6 @@ public class CoordActionInputCheckXComma
         }
     }
 
-    /* (non-Javadoc)
-     * @see org.apache.oozie.command.XCommand#getKey()
-     */
     @Override
     public String getKey(){
         return getName() + "_" + actionId;

Modified: oozie/trunk/release-log.txt
URL: 
http://svn.apache.org/viewvc/oozie/trunk/release-log.txt?rev=1502680&r1=1502679&r2=1502680&view=diff
==============================================================================
--- oozie/trunk/release-log.txt (original)
+++ oozie/trunk/release-log.txt Fri Jul 12 20:41:08 2013
@@ -5,6 +5,7 @@ OOZIE-1440 Build fails in certain enviro
 
 -- Oozie 4.0.0 release
 
+OOZIE-1451 CoordActionInputCheckX does a redundant eagerLoadState (rohini)
 OOZIE-1446 SLACalcStatus not updating the last modified time correctly and 
duplicate DURATION_* event (virag,mona via mona)
 OOZIE-1249 SLA Documentation (mona,virag,rohini via rohini)
 OOZIE-1441 Fix bugs related to coordchange and parentId in events (mona,virag 
via virag)


Reply via email to