Author: cwiklik
Date: Mon Sep 26 17:35:05 2011
New Revision: 1175967

URL: http://svn.apache.org/viewvc?rev=1175967&view=rev
Log:
UIMA-2239 removed synchronization on destinationMap. It is of type 
ConcurrentHashMap and doesnt need to be locked when iterating

Modified:
    
uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/controller/AggregateAnalysisEngineController_impl.java

Modified: 
uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/controller/AggregateAnalysisEngineController_impl.java
URL: 
http://svn.apache.org/viewvc/uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/controller/AggregateAnalysisEngineController_impl.java?rev=1175967&r1=1175966&r2=1175967&view=diff
==============================================================================
--- 
uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/controller/AggregateAnalysisEngineController_impl.java
 (original)
+++ 
uima/uima-as/trunk/uimaj-as-core/src/main/java/org/apache/uima/aae/controller/AggregateAnalysisEngineController_impl.java
 Mon Sep 26 17:35:05 2011
@@ -3110,10 +3110,9 @@ public class AggregateAnalysisEngineCont
     if ( buffer.length() == 0) {
       delegates.append("\n"+lbl1+getComponentName()+" Delegates:");
     }
-    synchronized(destinationMap) {
-      Set set = destinationMap.entrySet();
-      // For all delegates ...
-      for (Iterator it = set.iterator(); it.hasNext();) {
+    Set set = destinationMap.entrySet();
+    // For all delegates ...
+    for (Iterator it = set.iterator(); it.hasNext();) {
         delegates.append("\n");
         Map.Entry entry = (Map.Entry) it.next();
         Endpoint endpoint = (Endpoint) entry.getValue();
@@ -3141,8 +3140,6 @@ public class AggregateAnalysisEngineCont
             }
           }
         }
-      }
-      
     }
     buffer.append(delegates.toString());
   }


Reply via email to