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

Atul Sikaria commented on HADOOP-13508:
---------------------------------------

Looks reasonable - thanks Chris. 

Agree that the general parsing approach here should be looked at in the future, 
but this addresses current problem of not being able to parse valid permission 
strings.

> FsPermission's string constructor fails on valid permissions like "1777"
> ------------------------------------------------------------------------
>
>                 Key: HADOOP-13508
>                 URL: https://issues.apache.org/jira/browse/HADOOP-13508
>             Project: Hadoop Common
>          Issue Type: Bug
>            Reporter: Atul Sikaria
>            Assignee: Atul Sikaria
>         Attachments: HADOOP-13508-1.patch, HADOOP-13508-2.patch, 
> HADOOP-13508.003.patch, HADOOP-13508.004.patch, HADOOP-13508.005.patch, 
> HADOOP-13508.006.patch
>
>
> FsPermissions's string constructor breaks on valid permission strings, like 
> "1777". 
> This is because FsPermission class naïvely uses UmaskParser to do it’s 
> parsing of permissions: (from source code):
> public FsPermission(String mode) {
>     this((new UmaskParser(mode)).getUMask());
> }
> The mode string UMask accepts is subtly different (esp wrt sticky bit), so 
> parsing Umask is not the same as parsing FsPermission. 



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

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

Reply via email to