You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@hbase.apache.org by te...@apache.org on 2013/04/28 05:18:16 UTC

svn commit: r1476715 - /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java

Author: tedyu
Date: Sun Apr 28 03:18:15 2013
New Revision: 1476715

URL: http://svn.apache.org/r1476715
Log:
HBASE-8451 MetricsMBeanBase has concurrency issues in init (Liang Xie)


Modified:
    hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java

Modified: hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java?rev=1476715&r1=1476714&r2=1476715&view=diff
==============================================================================
--- hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java (original)
+++ hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java Sun Apr 28 03:18:15 2013
@@ -20,9 +20,9 @@
 package org.apache.hadoop.hbase.metrics;
 
 import java.util.ArrayList;
-import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
+import java.util.concurrent.ConcurrentHashMap;
 
 import javax.management.AttributeNotFoundException;
 import javax.management.MBeanAttributeInfo;
@@ -55,8 +55,8 @@ public class MetricsMBeanBase extends Me
   /** HBase MetricsBase implementations that MetricsDynamicMBeanBase does
    * not understand
    */
-  protected Map<String,MetricsBase> extendedAttributes =
-      new HashMap<String,MetricsBase>();
+  protected Map<String, MetricsBase> extendedAttributes =
+      new ConcurrentHashMap<String, MetricsBase>();
   protected MBeanInfo extendedInfo;
 
   protected MetricsMBeanBase( MetricsRegistry mr, String description ) {