Author: thomas.heute(a)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;