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

Chris Nauroth commented on HDFS-7773:
-------------------------------------

Hi, Anu.  This looks good.  I have just a few minor comments:

# {{DataXceiver}}: There is a new local variable named {{size}}.  However, I 
think this always ends up with the same value as the existing local variable 
{{read}}.  I suggest declaring {{long read = 0;}} in the outer scope instead of 
introducing a new variable.
# {{NameNodeMetrics}}: There are 2 lines of additional whitespace unrelated to 
{{totalFileOps}}.  I suggest removing those from the patch.
# {{TestNameNodeMetrics}}: Please check the formatting and indentation in this 
part of the patch.

Thanks!


> Additional metrics in HDFS to be accessed via jmx.
> --------------------------------------------------
>
>                 Key: HDFS-7773
>                 URL: https://issues.apache.org/jira/browse/HDFS-7773
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: datanode, namenode
>            Reporter: Anu Engineer
>            Assignee: Anu Engineer
>         Attachments: hdfs-7773.001.patch
>
>
> We would like to have the following metrics added to DataNode and name node 
> this to improve Ambari dashboard
> 1) DN disk i/o utilization
> 2) DN network i/o utilization
> 3) Namenode read operations 
> 4) Namenode write operations



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

Reply via email to