[GitHub] flink pull request: [FLINK-3180] [runtime] Log direct memory usage...

[uce]

Github user uce commented on the pull request:

    https://github.com/apache/flink/pull/1466#issuecomment-165524534
  
    I've added some related changes to this PR to display the direct memory 
usage in the web frontend as well. The current structure didn't make a lot of 
sense to me:
    
    ## Current
    
    ![screen shot 2015-12-17 at 17 28 
42](https://cloud.githubusercontent.com/assets/1756620/11876983/cf7b5280-a4ec-11e5-9a06-67075ad5a34a.png)
    
    ## This PR
    
    ![screen shot 2015-12-17 at 18 26 
00](https://cloud.githubusercontent.com/assets/1756620/11876991/dd456c98-a4ec-11e5-840c-1653fc393698.png)
    
    It's not that beautiful, but I think the ordering makes more sense now. 
I've also fixed a bug with the reporting of the PS-Scavenge GC times.
    
    (PS: Sorry for overloading this PR.)
    
    @mxm Is it OK if we address your comment in a follow up? I'm not sure how 
the respective bean is initialized and how expensive it is. In contrast to the 
other beans, this is not just a getter which initializes the bean on first 
access.


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---