[ 
https://issues.apache.org/jira/browse/HADOOP-4435?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12643140#action_12643140
 ] 

Amar Kamat commented on HADOOP-4435:
------------------------------------

bq. What if we display heap-details as part of cluster-summary. 
What I mean was to have _heap-info_ as a part of (i.e inside) the cluster 
summary table and not at the header but I wrongly showed it as a part of the JT 
info. Sorry for the confusion. So +1.

Few comments on the patch
1) I think changes to {{DatanodeDescriptor.java}} are unnecessary. 
2) Deprecate {{FsShell.byteDesc()}} and {{FsShell.limitDecimalTo2()}}. Point 
{{StringUtil.byteDesc()/StringUtil.limitDecimalTo2()}} in the comment.
3) Change the code that use {{FsShell.byteDesc()/FsShell.limitDecimalTo2}} to 
use {{StringUtils.byteDesc()/StringUtil.limitDecimalTo2()}}.
4) Do we need to show the heap info through cli?



> The JobTracker should display the amount of heap memory used
> ------------------------------------------------------------
>
>                 Key: HADOOP-4435
>                 URL: https://issues.apache.org/jira/browse/HADOOP-4435
>             Project: Hadoop Core
>          Issue Type: Improvement
>          Components: mapred
>            Reporter: dhruba borthakur
>            Assignee: dhruba borthakur
>            Priority: Minor
>             Fix For: 0.20.0
>
>         Attachments: clusterSummary.patch, clusterSummary.patch, 
> clusterSummary2.patch
>
>
> It would be nice to make the JobTracker UI display the amount of heap memory 
> currently in use. This is similar to the DFS UI that already displays the 
> amount if heap memory used by the NameNode.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to