svn commit: r1750405 - /uima/uima-ducc/trunk/uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/jd/scheduler/JdReservation.java

[degenaro]

Author: degenaro
Date: Mon Jun 27 19:08:34 2016
New Revision: 1750405

URL: http://svn.apache.org/viewvc?rev=1750405&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/jd/scheduler/JdReservation.java

Modified: 
uima/uima-ducc/trunk/uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/jd/scheduler/JdReservation.java
URL: 
http://svn.apache.org/viewvc/uima/uima-ducc/trunk/uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/jd/scheduler/JdReservation.java?rev=1750405&r1=1750404&r2=1750405&view=diff
==============================================================================
--- 
uima/uima-ducc/trunk/uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/jd/scheduler/JdReservation.java
 (original)
+++ 
uima/uima-ducc/trunk/uima-ducc-orchestrator/src/main/java/org/apache/uima/ducc/orchestrator/jd/scheduler/JdReservation.java
 Mon Jun 27 19:08:34 2016
@@ -143,8 +143,10 @@ public class JdReservation extends JdRes
                        synchronized(this) {
                                if(!map.containsKey(jdId)) {
                                        if(!isFull()) {
-                                               map.put(jdId, size);
-                                               retVal = getNodeIdentity();;
+                                               SizeBytes previous = 
map.putIfAbsent(jdId, size);
+                                               if(previous == null) {
+                                                       retVal = 
getNodeIdentity();
+                                               }
                                        }
                                }
                        }