[
https://issues.apache.org/jira/browse/HADOOP-2381?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12551078
]
Doug Cutting commented on HADOOP-2381:
--------------------------------------
I don't understand why 'loadPermission' is required. If a FileSystem wants to
be lazy about loading permissions, then it can subclass FileStatus, adding a
flag to indicate whether permissions have yet been loaded, and checking that
flag in its getStatus() implementation. The burden should be on the single
lazy implementation, not on the common API. If we find another implementation
that needs this too, then we could add a base class for lazy permissions, but
I'd rather keep FileStatus simpler.
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?
Finally, protected methods in public classes need javadoc.
> 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.
