Author: degenaro
Date: Tue Jun 28 10:06:15 2016
New Revision: 1750461

URL: http://svn.apache.org/viewvc?rev=1750461&view=rev
Log:
UIMA-4989 Fix problems found by FindBugs

Modified:
    
uima/uima-ducc/trunk/uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/StateManager.java

Modified: 
uima/uima-ducc/trunk/uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/StateManager.java
URL: 
http://svn.apache.org/viewvc/uima/uima-ducc/trunk/uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/StateManager.java?rev=1750461&r1=1750460&r2=1750461&view=diff
==============================================================================
--- 
uima/uima-ducc/trunk/uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/StateManager.java
 (original)
+++ 
uima/uima-ducc/trunk/uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/StateManager.java
 Tue Jun 28 10:06:15 2016
@@ -247,7 +247,16 @@ public class StateManager {
                while(workMapIterator.hasNext()) {
                        DuccId duccId = workMapIterator.next();
                        IDuccWork duccWork = 
WorkMapHelper.findDuccWork(workMap, duccId, this, methodName);
-                       switch(duccWork.getDuccType()) {
+                       if(duccWork == null) {
+                               logger.warn(methodName, duccId, "not found");
+                               continue;
+                       }
+                       DuccType duccType = duccWork.getDuccType();
+                       if(duccType == null) {
+                               logger.warn(methodName, duccId, "no type?");
+                               continue;
+                       }
+                       switch(duccType) {
                        case Job:
                        case Service:
                                DuccWorkJob duccWorkJob = (DuccWorkJob)duccWork;


Reply via email to