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

Ayush Saxena commented on HDFS-13377:
-------------------------------------

Thanx [~elgoiri] for taking a look.
for ugi,doAs do you mean something like this ? 

{code:java}
 ugi.doAs((PrivilegedExceptionAction<Object>) () -> {
      assertEquals("Not running as new user", userName2,
          UserGroupInformation.getCurrentUser().getShortUserName());
      DFSAdmin userAdmin = new DFSAdmin(conf);

      String[] args2 = new String[]{"-setQuota", "5", subDir.toString()};
      TestQuota.runCommand(userAdmin, args2, true);
      args2 = new String[]{"-setSpaceQuota", "64", subDir.toString()};
      TestQuota.runCommand(userAdmin, args2, true);
      return null;
    });
{code}
 
[~hadoop_yangyun]

* Please remove the unused imports from the test.
* Add Comments above each test, as what they are testing.
* Typo here :
{code:java}
 assertEquals("Not same with settin space quota",
{code}
* Would be good, if we have a test, checking owner isn't allowed to setQuota 
when this configuration isn't enabled.


> The owner of folder can set quota for his sub folder
> ----------------------------------------------------
>
>                 Key: HDFS-13377
>                 URL: https://issues.apache.org/jira/browse/HDFS-13377
>             Project: Hadoop HDFS
>          Issue Type: Improvement
>          Components: namenode
>            Reporter: Yang Yun
>            Assignee: Yang Yun
>            Priority: Minor
>         Attachments: HDFS-13377.003.patch, HDFS-13377.patch, 
> HDFS-13377.patch, HDFS-13377.patch
>
>
> Currently, only  super user can set quota. That is huge burden for 
> administrator in a large system. Add a new feature to let the owner of a 
> folder also has the privilege to set quota for his sub folders. 



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to