Skip to content

Commit 58275bb

Browse files
jinggouvirajjasani
authored andcommitted
HBASE-27892 Report memstore on-heap and off-heap size as jmx metrics (#5308)
Signed-off-by: Viraj Jasani <[email protected]>
1 parent 8b0dfc1 commit 58275bb

File tree

3 files changed

+10
-0
lines changed

3 files changed

+10
-0
lines changed

hbase-hadoop-compat/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsRegionServerSource.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -238,6 +238,10 @@ public interface MetricsRegionServerSource extends BaseSource, JvmPauseMonitorSo
238238
String MAX_COMPACTED_STORE_FILE_REF_COUNT = "maxCompactedStoreFileRefCount";
239239
String MEMSTORE_SIZE = "memStoreSize";
240240
String MEMSTORE_SIZE_DESC = "Size of the memstore";
241+
String MEMSTORE_HEAP_SIZE = "memStoreHeapSize";
242+
String MEMSTORE_HEAP_SIZE_DESC = "On-heap Size of the memstore";
243+
String MEMSTORE_OFFHEAP_SIZE = "memStoreOffHeapSize";
244+
String MEMSTORE_OFFHEAP_SIZE_DESC = "Off-heap Size of the memstore";
241245
String STOREFILE_SIZE = "storeFileSize";
242246
String STOREFILE_SIZE_GROWTH_RATE = "storeFileSizeGrowthRate";
243247
String MAX_STORE_FILE_AGE = "maxStoreFileAge";

hbase-hadoop-compat/src/main/java/org/apache/hadoop/hbase/regionserver/MetricsRegionServerSourceImpl.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -520,6 +520,10 @@ private MetricsRecordBuilder addGaugesToMetricsRecordBuilder(MetricsRecordBuilde
520520
.addGauge(Interns.info(MAX_STOREFILE_COUNT, MAX_STOREFILE_COUNT_DESC),
521521
rsWrap.getMaxStoreFiles())
522522
.addGauge(Interns.info(MEMSTORE_SIZE, MEMSTORE_SIZE_DESC), rsWrap.getMemStoreSize())
523+
.addGauge(Interns.info(MEMSTORE_HEAP_SIZE, MEMSTORE_HEAP_SIZE_DESC),
524+
rsWrap.getOnHeapMemStoreSize())
525+
.addGauge(Interns.info(MEMSTORE_OFFHEAP_SIZE, MEMSTORE_OFFHEAP_SIZE_DESC),
526+
rsWrap.getOffHeapMemStoreSize())
523527
.addGauge(Interns.info(STOREFILE_SIZE, STOREFILE_SIZE_DESC), rsWrap.getStoreFileSize())
524528
.addGauge(Interns.info(STOREFILE_SIZE_GROWTH_RATE, STOREFILE_SIZE_GROWTH_RATE_DESC),
525529
rsWrap.getStoreFileSizeGrowthRate())

hbase-server/src/test/java/org/apache/hadoop/hbase/regionserver/TestMetricsRegionServer.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -83,6 +83,8 @@ public void testWrapperSource() {
8383
HELPER.assertGauge("hlogFileSize", 1024000, serverSource);
8484
HELPER.assertGauge("storeFileCount", 300, serverSource);
8585
HELPER.assertGauge("memstoreSize", 1025, serverSource);
86+
HELPER.assertGauge("memstoreHeapSize", 500, serverSource);
87+
HELPER.assertGauge("memstoreOffHeapSize", 600, serverSource);
8688
HELPER.assertGauge("storeFileSize", 1900, serverSource);
8789
HELPER.assertGauge("storeFileSizeGrowthRate", 50.0, serverSource);
8890
HELPER.assertCounter("totalRequestCount", 899, serverSource);

0 commit comments

Comments
 (0)