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

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


>Also, I don't understand the comment 'null in case of error'. If the default 
>value is non-null, and the setter does not permit setting to a null value, 
>then the value could never be null. Wouldn't that be better?

I don't have strong opinion either side. Question Nicholas raised was how does 
user know if there was some error in getting these values? Two options I could 
think of were null return or IOException. Another option could be to add a 
error flag that a user can check if required. Let me know.

> I don't understand why 'loadPermission' is required. 

This is strictly not required. I don't see the extra burden on base class 
either (apart from check for null). Sure, we can change it to using a flag and 
make the set methods not permit 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, 
> HADOOP-2381.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