Apache
/
hadoop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

HADOOP-15067. GC time percentage reported in JvmMetrics should be a gauge, not counter. Contributed by Misha Dmitriev. 

(cherry picked from commit d162252d7a)
This commit is contained in:
Xiao Chen 2017-11-23 09:00:59 -08:00
parent 5884b8a80a
commit 4f71c27f2e
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
hadoop-common-project/hadoop-common/src/main/java/org/apache/hadoop/metrics2/source/JvmMetrics.java
View File

@ -188,7 +188,7 @@ public class JvmMetrics implements MetricsSource {
} }
if (gcTimeMonitor != null) { if (gcTimeMonitor != null) {
rb.addCounter(GcTimePercentage, rb.addGauge(GcTimePercentage,
gcTimeMonitor.getLatestGcData().getGcTimePercentage()); gcTimeMonitor.getLatestGcData().getGcTimePercentage());
} }
} }

2
hadoop-common-project/hadoop-common/src/test/java/org/apache/hadoop/metrics2/source/TestJvmMetrics.java
View File

@ -101,7 +101,7 @@ public class TestJvmMetrics {
verify(rb).tag(SessionId, "test"); verify(rb).tag(SessionId, "test");
for (JvmMetricsInfo info : JvmMetricsInfo.values()) { for (JvmMetricsInfo info : JvmMetricsInfo.values()) {
if (info.name().equals("GcTimePercentage")) { if (info.name().equals("GcTimePercentage")) {
verify(rb).addCounter(eq(info), anyInt()); verify(rb).addGauge(eq(info), anyInt());
} }
} }
} }