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

Raghu Angadi commented on HADOOP-2381:
--------------------------------------


will remove 's' from {{getPermissions()}} etc.

> Since we cannot make FsPermission immutable, is it better to return a new 
> object every time? Otherwise, it is a security hole.
I was hoping 'final' will some how prevent it at runtime. I will make it throw 
RuntimeException if someone invokes readfields one this default object. Would 
that be ok?

> In RawLocalFileSystem.loadPermissions(), should it throws IOException (or 
> return null) if there is any exception? Otherwise, we cannot tell whether the 
> permission returned is real or just the default because the command failed.

Should getOwner() etc throw the IOException or return null?



> Support permission information in FileStatus
> --------------------------------------------
>
>                 Key: HADOOP-2381
>                 URL: https://issues.apache.org/jira/browse/HADOOP-2381
>             Project: Hadoop
>          Issue Type: New Feature
>          Components: fs
>    Affects Versions: 0.15.0
>            Reporter: Tsz Wo (Nicholas), SZE
>            Assignee: Raghu Angadi
>             Fix For: 0.16.0
>
>         Attachments: 2381_20071207.patch, HADOOP-2381.patch
>
>
> In HADOOP-2288,  FileSystem API is changed to support access control.  
> FileStatus should also be changed to support permission information.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to