You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by st...@apache.org on 2018/04/14 15:30:42 UTC
hbase git commit: HBASE-20233 Remove redundant region server metric
Repository: hbase
Updated Branches:
refs/heads/branch-2 f5d970eba -> f154afa7c
HBASE-20233 Remove redundant region server metric
Remove unused import
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/f154afa7
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/f154afa7
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/f154afa7
Branch: refs/heads/branch-2
Commit: f154afa7cd7d7f41e1ec1553be56a320bfcf1cde
Parents: f5d970e
Author: Xu Cang <xc...@salesforce.com>
Authored: Fri Apr 13 15:56:32 2018 -0700
Committer: Michael Stack <st...@apache.org>
Committed: Sat Apr 14 08:30:32 2018 -0700
----------------------------------------------------------------------
.../hadoop/hbase/regionserver/MetricsRegionAggregateSource.java | 1 -
.../hbase/regionserver/MetricsRegionAggregateSourceImpl.java | 2 --
2 files changed, 3 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/f154afa7/hbase-hadoop-compat/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsRegionAggregateSource.java
----------------------------------------------------------------------
diff --git a/hbase-hadoop-compat/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsRegionAggregateSource.java b/hbase-hadoop-compat/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsRegionAggregateSource.java
index 23d0259..1d9e1ac 100644
--- a/hbase-hadoop-compat/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsRegionAggregateSource.java
+++ b/hbase-hadoop-compat/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsRegionAggregateSource.java
@@ -48,7 +48,6 @@ public interface MetricsRegionAggregateSource extends BaseSource {
*/
String METRICS_JMX_CONTEXT = "RegionServer,sub=" + METRICS_NAME;
- String NUM_REGIONS = "numRegions";
String NUMBER_OF_REGIONS_DESC = "Number of regions in the metrics system";
/**
http://git-wip-us.apache.org/repos/asf/hbase/blob/f154afa7/hbase-hadoop2-compat/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsRegionAggregateSourceImpl.java
----------------------------------------------------------------------
diff --git a/hbase-hadoop2-compat/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsRegionAggregateSourceImpl.java b/hbase-hadoop2-compat/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsRegionAggregateSourceImpl.java
index cb78ccf..96c1a2b 100644
--- a/hbase-hadoop2-compat/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsRegionAggregateSourceImpl.java
+++ b/hbase-hadoop2-compat/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsRegionAggregateSourceImpl.java
@@ -27,7 +27,6 @@ import org.apache.yetus.audience.InterfaceAudience;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.apache.hadoop.hbase.metrics.BaseSourceImpl;
-import org.apache.hadoop.hbase.metrics.Interns;
import org.apache.hadoop.metrics2.MetricsCollector;
import org.apache.hadoop.metrics2.MetricsRecordBuilder;
import org.apache.hadoop.metrics2.impl.JmxCacheBuster;
@@ -105,7 +104,6 @@ public class MetricsRegionAggregateSourceImpl extends BaseSourceImpl
((MetricsRegionSourceImpl) regionMetricSource).snapshot(mrb, all);
}
}
- mrb.addGauge(Interns.info(NUM_REGIONS, NUMBER_OF_REGIONS_DESC), regionSources.size());
metricsRegistry.snapshot(mrb, all);
}
}