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

[sohami]

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

    https://github.com/apache/drill/pull/1203#discussion_r181161557
  
    --- 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 --
    
    please replace `registerAll` with `register`. See comment above in 
`CpuGaugeSet`


---