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:12:19 UTC
svn commit: r1476713 -
/hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java
Author: tedyu
Date: Sun Apr 28 03:12:19 2013
New Revision: 1476713
URL: http://svn.apache.org/r1476713
Log:
HBASE-8451 MetricsMBeanBase has concurrency issues in init (Liang Xie)
Modified:
hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java
Modified: hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java
URL: http://svn.apache.org/viewvc/hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java?rev=1476713&r1=1476712&r2=1476713&view=diff
==============================================================================
--- hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java (original)
+++ hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java Sun Apr 28 03:12:19 2013
@@ -19,9 +19,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;
@@ -57,8 +57,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 ) {