[gatein-commits] gatein SVN: r5203 - epp/portal/branches/EPP_5_1_Branch/webui/framework/src/main/java/org/exoplatform/webui/application.

[do-not-reply at jboss.org]

Author: thomas.heute at jboss.com
Date: 2010-11-22 09:41:04 -0500 (Mon, 22 Nov 2010)
New Revision: 5203

Modified:
   epp/portal/branches/EPP_5_1_Branch/webui/framework/src/main/java/org/exoplatform/webui/application/ConfigurationManager.java
Log:
JBEPP-577: Concurrency problem (HashMap used in class ConfigurationManager in multithreaded environment)
Rolledback, replaced by JBEPP-657


Modified: epp/portal/branches/EPP_5_1_Branch/webui/framework/src/main/java/org/exoplatform/webui/application/ConfigurationManager.java
===================================================================
--- epp/portal/branches/EPP_5_1_Branch/webui/framework/src/main/java/org/exoplatform/webui/application/ConfigurationManager.java	2010-11-22 14:16:31 UTC (rev 5202)
+++ epp/portal/branches/EPP_5_1_Branch/webui/framework/src/main/java/org/exoplatform/webui/application/ConfigurationManager.java	2010-11-22 14:41:04 UTC (rev 5203)
@@ -40,7 +40,6 @@
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
-import java.util.concurrent.ConcurrentHashMap;
 
 /**
  * May 10, 2006
@@ -56,16 +55,8 @@
     * <p/>
     * The components of which we manage the configuration
     */
-   //private Map<String, Component> configs_ = new HashMap<String, Component>();
+   private Map<String, Component> configs_ = new HashMap<String, Component>();
 
-   /**
-    * Minh Hoang TO: First attempt to synchronize the map, we simply replace HashMap with ConcurrentHashMap
-    * and default values for load factor, initial capacity and concurrentcyLevel
-    * 
-    * TODO: Need to examine the performance influence in the future for a better synchronizing 
-    */
-   private Map<String, Component> configs_ = new ConcurrentHashMap<String, Component>();
-   
    /** The logger. */
    private final Logger log;