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

Charles Lamb commented on HDFS-7390:
------------------------------------

Hi [~benoyantony],

Thanks for working on this -- it's a good improvement.

I'm currently working on HDFS-7331. In 
https://issues.apache.org/jira/browse/HDFS-7331?focusedCommentId=14195484&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-14195484,
 [~wheat9] suggested that I just return the Map directly rather than the 
JSON.toString. If I change your code to do that, the output looks like this and 
you'd be returning a JSON object, not just a string:

{noformat}
    "TieredStorageStats" : "{DISK={capacityTotal=42275598336, 
capacityUsed=24576, capacityRemaining=21051469824, blockPoolUsed=24576, 
nodesInService=1}}",
{noformat}

Would that work?

If you end up keeping JsonString, I assume you'll use the version from 
HADOOP-11308's second patch and not the one in the HDFS-7390 patch. That would 
probably happen just by virtue of commit order of the two patches. Also, there 
are several lines that are > 80 chars.



> Provide JMX metrics per storage type
> ------------------------------------
>
>                 Key: HDFS-7390
>                 URL: https://issues.apache.org/jira/browse/HDFS-7390
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>    Affects Versions: 2.5.2
>            Reporter: Benoy Antony
>            Assignee: Benoy Antony
>         Attachments: HDFS-7390.patch, HDFS-7390.patch
>
>
> HDFS-2832  added heterogeneous support. In a cluster with different storage 
> types, it is useful to have metrics per storage type. 



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

Reply via email to