diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2022-04-27 15:51:09 +0200 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2022-04-27 15:51:09 +0200 |
commit | 2b0994346361e2e292dc390abd21129ba527f39f (patch) | |
tree | 922d718015f270b8820d0315bc2f296ac8256061 /container-disc | |
parent | a69561e12789d506c3b45f81ffda4baae05b3965 (diff) |
Report metrics for the size of the native heap.
Diffstat (limited to 'container-disc')
-rw-r--r-- | container-disc/src/main/java/com/yahoo/container/jdisc/metric/MetricUpdater.java | 13 | ||||
-rw-r--r-- | container-disc/src/test/java/com/yahoo/container/jdisc/metric/MetricUpdaterTest.java | 2 |
2 files changed, 14 insertions, 1 deletions
diff --git a/container-disc/src/main/java/com/yahoo/container/jdisc/metric/MetricUpdater.java b/container-disc/src/main/java/com/yahoo/container/jdisc/metric/MetricUpdater.java index 62982d66978..4659c2acc36 100644 --- a/container-disc/src/main/java/com/yahoo/container/jdisc/metric/MetricUpdater.java +++ b/container-disc/src/main/java/com/yahoo/container/jdisc/metric/MetricUpdater.java @@ -5,6 +5,8 @@ import com.google.inject.Inject; import com.yahoo.component.AbstractComponent; import com.yahoo.jdisc.Metric; import com.yahoo.jdisc.statistics.ContainerWatchdogMetrics; +import com.yahoo.nativec.NativeHeap; + import java.lang.management.BufferPoolMXBean; import java.lang.management.ManagementFactory; import java.nio.file.DirectoryStream; @@ -24,6 +26,9 @@ import java.util.TimerTask; */ public class MetricUpdater extends AbstractComponent { + private static final String NATIVE_FREE_MEMORY_BYTES = "mem.native.free"; + private static final String NATIVE_USED_MEMORY_BYTES = "mem.native.used"; + private static final String NATIVE_TOTAL_MEMORY_BYTES = "mem.native.total"; private static final String HEAP_FREE_MEMORY_BYTES = "mem.heap.free"; private static final String HEAP_USED_MEMORY_BYTES = "mem.heap.used"; private static final String HEAP_TOTAL_MEMORY_BYTES = "mem.heap.total"; @@ -116,6 +121,13 @@ public class MetricUpdater extends AbstractComponent { metric.set(DIRECT_COUNT, count, null); } + private void nativeHeapUsed() { + NativeHeap nativeHeap = NativeHeap.sample(); + metric.set(NATIVE_FREE_MEMORY_BYTES, nativeHeap.availableSize(), null); + metric.set(NATIVE_USED_MEMORY_BYTES, nativeHeap.usedSize(), null); + metric.set(NATIVE_TOTAL_MEMORY_BYTES, nativeHeap.totalSize(), null); + } + @Override public void run() { long freeMemory = runtime.freeMemory(); @@ -127,6 +139,7 @@ public class MetricUpdater extends AbstractComponent { metric.set(MEMORY_MAPPINGS_COUNT, count_mappings(), null); metric.set(OPEN_FILE_DESCRIPTORS, count_open_files(), null); directMemoryUsed(); + nativeHeapUsed(); containerWatchdogMetrics.emitMetrics(metric); garbageCollectionMetrics.emitMetrics(metric); diff --git a/container-disc/src/test/java/com/yahoo/container/jdisc/metric/MetricUpdaterTest.java b/container-disc/src/test/java/com/yahoo/container/jdisc/metric/MetricUpdaterTest.java index 68d73a7914a..f49ccf2c2f6 100644 --- a/container-disc/src/test/java/com/yahoo/container/jdisc/metric/MetricUpdaterTest.java +++ b/container-disc/src/test/java/com/yahoo/container/jdisc/metric/MetricUpdaterTest.java @@ -27,7 +27,7 @@ public class MetricUpdaterTest { ContainerWatchdogMetrics containerWatchdogMetrics = mock(ContainerWatchdogMetrics.class); new MetricUpdater(new MockScheduler(), metric, containerWatchdogMetrics); verify(containerWatchdogMetrics, times(1)).emitMetrics(any()); - verify(metric, times(9 + 2 * gcCount)).set(anyString(), any(), any()); + verify(metric, times(12 + 2 * gcCount)).set(anyString(), any(), any()); } private static class MockScheduler implements MetricUpdater.Scheduler { |