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

Haohui Mai commented on HDFS-8115:
----------------------------------

The id of user / group only makes sense for a particular run of NN only.

This is an implementation detail and it should not be exposed to the public.




> Make PermissionStatusFormat public
> ----------------------------------
>
>                 Key: HDFS-8115
>                 URL: https://issues.apache.org/jira/browse/HDFS-8115
>             Project: Hadoop HDFS
>          Issue Type: Bug
>            Reporter: Arun Suresh
>            Assignee: Arun Suresh
>            Priority: Minor
>         Attachments: HDFS-8115.1.patch
>
>
> implementations of {{INodeAttributeProvider}} are required to provide an 
> implementation of {{getPermissionLong()}} method. Unfortunately, the long 
> permission format is an encoding of the user, group and mode with each field 
> converted to int using {{SerialNumberManager}} which is package protected.
> Thus it would be nice to make the {{PermissionStatusFormat}} enum public (and 
> also make the {{toLong()}} static method public) so that user specified 
> implementations of {{INodeAttributeProvider}} may use it.
> This would also make it more consistent with {{AclStatusFormat}} which I 
> guess has been made public for the same reason.



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

Reply via email to