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

Ayush Saxena commented on HDFS-14210:
-------------------------------------

LOCAL shall be having one only. I guess. That is not a part of pathAll() too.

{code:java}
  public boolean isAll() {
    DestinationOrder order = getDestOrder();
    return order == DestinationOrder.HASH_ALL ||
        order == DestinationOrder.RANDOM ||
        order == DestinationOrder.SPACE;
{code}

 LOCAL is like the local destination shall be tried first as it is written. So 
since that isn't part of isPathAll(). It shall also be having in only one as 
what I check in mkdir if not part of isPathAll() then call is single only.

So, To our implementation. if use the approach set for setOwner. Checking 
isPathAll() shall be correct for all. But just not on HASH and LOCAL if we call 
the operation on the mount only. Else that is correct. That is why we can check 
if that is mount entry or not.

> RBF: ModifyACL should work over all the destinations
> ----------------------------------------------------
>
>                 Key: HDFS-14210
>                 URL: https://issues.apache.org/jira/browse/HDFS-14210
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>            Reporter: Shubham Dewan
>            Assignee: Shubham Dewan
>            Priority: Major
>         Attachments: HDFS-14210-HDFS-13891.002.patch, 
> HDFS-14210-HDFS-13891.003.patch, HDFS-14210.001.patch
>
>
> 1) A mount point with multiple destinations.
> 2) ./bin/hdfs dfs -setfacl -m user:abc:rwx /testacl
> 3) where /testacl => /test1, /test2
> 4) command works for only one destination.
> ACL should be set on both of the destinations.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to