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

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

Github user iampeter commented on the pull request:

    https://github.com/apache/flink/pull/1236#issuecomment-146333800
  
    @sachingoel0101 @aljoscha looks great, but I would make the following 
changes if possible:
    
    1. Move the 'livechart' directive to a separate file - 
`taskmanager.dir.coffee`
    2. Move the Highcharts config in `index.coffee` to either `.run()` in 
`index.coffee`, or the directive
    3. Remove the 'No such task manager exits' text that blinks when the task 
manager details are loading
    
    Thanks,
    Piotr
    



> Add CPU/Network utilization graphs to new web dashboard
> -------------------------------------------------------
>
>                 Key: FLINK-2730
>                 URL: https://issues.apache.org/jira/browse/FLINK-2730
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Webfrontend
>    Affects Versions: 0.10
>            Reporter: Stephan Ewen
>            Assignee: Sachin Goel
>             Fix For: 0.10
>
>
> The charts rendered in the previous dashboard should be added to the new web 
> dashboard.



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

Reply via email to