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

Haohui Mai commented on HDFS-7476:
----------------------------------

Thanks very much for the reviews. The v3 patch updates {{TestAuditLogger}}.

I might there might be a misunderstanding in 
{{TestNameNodeAcl#testGetAclStatusRequiresTraverseOrSuper}}. Traverse in 
{{FSPermissionChecker}} means that the the user can access every directory 
along the path except for the file. We might need to change it to 
{{checkAccess}}. Thoughts?

> Consolidate ACL-related operations to a single class
> ----------------------------------------------------
>
>                 Key: HDFS-7476
>                 URL: https://issues.apache.org/jira/browse/HDFS-7476
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Haohui Mai
>            Assignee: Haohui Mai
>         Attachments: HDFS-7476.000.patch, HDFS-7476.001.patch, 
> HDFS-7476.002.patch, HDFS-7476.003.patch
>
>
> This jira proposes to consolidate ACL-related operations scattered in 
> {{FSNamesystem}} and {{FSDirectory}} into a single class.



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

Reply via email to