[ 
https://issues.apache.org/jira/browse/HDFS-5624?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Stephen Chu updated HDFS-5624:
------------------------------

    Attachment: HDFS-5624.001.patch

Attaching a patch with ACL + ViewFs tests. Thanks, Chris, for the guidance. To 
test these changes, I ran all the hadoop-common and hadoop-hdfs viewfs tests 
successfully.

Changes:

* For the modifying ACL operations in ViewFileSystem.java, if the target path 
is an internal dir, then we throw the "InternalDir of ViewFileSystem is 
readonly" AccessControlException. This makes the ACL operations consistent with 
the other methods like setPermission. Previously, we would throw an 
UnsupportedOperationException.

* In the case of getAclStatus, we made changes to make it consistent to 
getFileStatus in the case of an internal dir. User is set to the FileSystem 
user's name, group is set to the FileSystem user's primary group (first group), 
sticky bit is false, and ACL entries are equivalent to 555 permissions.

* Add ACL tests for internal dir paths in ViewFileSystemBaseTest.java

* Add TestViewFsWithAcls which brings up a federated MiniDFSCluster with 2 
NameNodes and verifies that ACL operations + ViewFs are directed to the correct 
NameNodes.



> Add tests for ACLs in combination with viewfs.
> ----------------------------------------------
>
>                 Key: HDFS-5624
>                 URL: https://issues.apache.org/jira/browse/HDFS-5624
>             Project: Hadoop HDFS
>          Issue Type: Test
>          Components: hdfs-client
>    Affects Versions: 2.4.0
>            Reporter: Chris Nauroth
>            Assignee: Stephen Chu
>         Attachments: HDFS-5624.001.patch
>
>
> Add tests verifying that in a federated deployment, a viewfs wrapped over 
> multiple federated NameNodes will dispatch the ACL operations to the correct 
> NameNode.



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to