[ 
https://issues.apache.org/jira/browse/FLINK-5209?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15707693#comment-15707693
 ] 

ASF GitHub Bot commented on FLINK-5209:
---------------------------------------

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

    https://github.com/apache/flink/pull/2902#discussion_r90174556
  
    --- Diff: 
flink-runtime-web/src/main/java/org/apache/flink/runtime/webmonitor/handlers/TaskManagersHandler.java
 ---
    @@ -124,13 +124,21 @@ public String handleJsonRequest(Map<String, String> 
pathParams, Map<String, Stri
                                                        
gen.writeNumberField("totalUsed", heapUsed + nonHeapUsed);
                                                        
gen.writeNumberField("totalMax", heapTotal + nonHeapTotal);
     
    -                                                   
gen.writeStringField("directCount", 
metrics.getMetric("Status.JVM.Memory.Direct.Count", "0"));
    -                                                   
gen.writeStringField("directUsed", 
metrics.getMetric("Status.JVM.Memory.Direct.MemoryUsed", "0"));
    -                                                   
gen.writeStringField("directMax", 
metrics.getMetric("Status.JVM.Memory.Direct.TotalCapacity", "0"));
    +                                                   long directCount = 
Long.valueOf(metrics.getMetric("Status.JVM.Memory.direct.Count", "0"));
    --- End diff --
    
    is there a benefit to passing them as longs?


> Fix TaskManager metrics
> -----------------------
>
>                 Key: FLINK-5209
>                 URL: https://issues.apache.org/jira/browse/FLINK-5209
>             Project: Flink
>          Issue Type: Bug
>          Components: Webfrontend
>    Affects Versions: 1.2.0
>            Reporter: Greg Hogan
>            Assignee: Greg Hogan
>             Fix For: 1.2.0
>
>
> Properly propagate the network and non-JVM memory metrics to the web UI.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to