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

Xiaoyu Yao commented on HDFS-9234:
----------------------------------

Thanks [~surendrasingh] for working on this. The patch looks good to me 
overall. Just one question:

I don't think it is right to cast the type quota and consumed from long to 
integer below. Is there a particular reason for that?
{code}
0               contentSummaryBuilder = contentSummaryBuilder.typeQuota(t,
331                 (Integer) type.get("quota")).typeConsumed(t,
332                 (Integer) type.get("consumed"));
{code}

> WebHdfs : getContentSummary() should give quota for storage types
> -----------------------------------------------------------------
>
>                 Key: HDFS-9234
>                 URL: https://issues.apache.org/jira/browse/HDFS-9234
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: webhdfs
>    Affects Versions: 2.7.1
>            Reporter: Surendra Singh Lilhore
>            Assignee: Surendra Singh Lilhore
>         Attachments: HDFS-9234-001.patch, HDFS-9234-002.patch
>
>
> Currently webhdfs API for ContentSummary give only namequota and spacequota 
> but it will not give storage types quota.



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

Reply via email to