Skip to content

Commit

Permalink
HBASE-27713 Remove numRegions in Region Metrics (#5107)
Browse files Browse the repository at this point in the history
Signed-off-by: Duo Zhang <[email protected]>
  • Loading branch information
thangTang authored Apr 8, 2023
1 parent f36587c commit 251e498
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -47,9 +47,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";

/**
* Register a MetricsRegionSource as being open.
* @param source the source for the region being opened.
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -22,7 +22,6 @@
import java.util.concurrent.ConcurrentHashMap;
import java.util.concurrent.TimeUnit;
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;
Expand Down Expand Up @@ -99,7 +98,6 @@ public void getMetrics(MetricsCollector collector, boolean all) {
((MetricsRegionSourceImpl) regionMetricSource).snapshot(mrb, all);
}
}
mrb.addGauge(Interns.info(NUM_REGIONS, NUMBER_OF_REGIONS_DESC), regionSources.size());
metricsRegistry.snapshot(mrb, all);
}
}
Expand Down

0 comments on commit 251e498

Please sign in to comment.