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()); }