[GitHub] drill pull request #1203: DRILL-6289: Cluster view should show more relevant...

[kkhatua]

Github user kkhatua commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1203#discussion_r182501230
  
    --- Diff: 
common/src/main/java/org/apache/drill/exec/metrics/DrillMetrics.java ---
    @@ -60,6 +61,7 @@ private static void registerSystemMetrics() {
           REGISTRY.registerAll(new 
BufferPoolMetricSet(ManagementFactory.getPlatformMBeanServer()));
           REGISTRY.registerAll(new MemoryUsageGaugeSet());
           REGISTRY.registerAll(new ThreadStatesGaugeSet());
    +      REGISTRY.registerAll(new 
CpuGaugeSet(ManagementFactory.getOperatingSystemMXBean()));
    --- End diff --
    
    Retained the GaugeSet to carry ProcessCPU and Uptime information as well. 
So, I've not replaced `registerAll`.


---