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

Matt Foley commented on HADOOP-7249:
------------------------------------

Hi Daryn, generally looks good but in FsShellPermissions.parseOwnerGroup() a 
tab (instead of spaces) crept in to the added lines just below "group = null;", 
to wit:
      if (group != null && group.length() == 0) {
-        group = null;
-      }
+          group = null;
+          } <-- problem here, and possibly prev line

> Refactor FsShell's chmod/chown/chgrp
> ------------------------------------
>
>                 Key: HADOOP-7249
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7249
>             Project: Hadoop Common
>          Issue Type: Improvement
>          Components: fs
>    Affects Versions: 0.23.0
>            Reporter: Daryn Sharp
>            Assignee: Daryn Sharp
>         Attachments: HADOOP-7249.patch
>
>
> Need to refactor permissions commands to conform to new FsCommand class.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to