hadoop-hdfs-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Xiaoyu Yao (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HDFS-11770) Ozone: KSM: Add setVolumeProperty
Date Sat, 20 May 2017 12:40:04 GMT

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

Xiaoyu Yao commented on HDFS-11770:

Thanks [~msingh] for the update with additional tests. The patch looks pretty good to me.
Here a few minor issues:

NIT: line 70 extra "the" can be removed.

NIT: line 210 same as above

NIT: line 296 same as above

Line 107-111: in delVolumeFromOwnerList() 
when all the volume of a user is deleted, should we remove the empty user entry instead of

updating the empty user entry? 

Line 164-167: We should set the Quota in the VolumeInfo returned here
to validate the SetQuota that has been implemented here.

Can we add test for SetQuota with the getVolumeInfo support?

Checkstyle from latest Jenkins.

> Ozone: KSM: Add setVolumeProperty
> ---------------------------------
>                 Key: HDFS-11770
>                 URL: https://issues.apache.org/jira/browse/HDFS-11770
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ozone
>            Reporter: Anu Engineer
>            Assignee: Mukul Kumar Singh
>             Fix For: HDFS-7240
>         Attachments: HDFS-11770-HDFS-7240.001.patch, HDFS-11770-HDFS-7240.002.patch,
HDFS-11770-HDFS-7240.003.patch, HDFS-11770-HDFS-7240.005.patch
> SetVolumeProperty allows Ozone administrators to change the ownership of a volume and

This message was sent by Atlassian JIRA

To unsubscribe, e-mail: hdfs-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-help@hadoop.apache.org

View raw message