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

Posted to commits@hbase.apache.org by ns...@apache.org on 2011/10/11 04:08:59 UTC

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

Author: nspiegelberg
Date: Tue Oct 11 02:08:58 2011
New Revision: 1181444

URL: http://svn.apache.org/viewvc?rev=1181444&view=rev
Log:
Fixed false error message from MetricsUtil. Reviewed by: Kannan

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

Modified: hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java
URL: http://svn.apache.org/viewvc/hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java?rev=1181444&r1=1181443&r2=1181444&view=diff
==============================================================================
--- hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java (original)
+++ hbase/branches/0.89/src/main/java/org/apache/hadoop/hbase/metrics/MetricsMBeanBase.java Tue Oct 11 02:08:58 2011
@@ -71,7 +71,7 @@ public class MetricsMBeanBase extends Me
   private static MetricsRegistry copyMinusHBaseMetrics(final MetricsRegistry mr) {
     MetricsRegistry copy = new MetricsRegistry();
     for (MetricsBase metric : mr.getMetricsList()) {
-      if (metric instanceof org.apache.hadoop.hbase.metrics.MetricsRate) {
+      if (metric instanceof MetricsRate || metric instanceof MetricsString) {
         continue;
       }
       copy.add(metric.getName(), metric);