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

Tsz Wo Nicholas Sze commented on HDFS-6326:
-------------------------------------------

>  Suppose a 2.3.0 process attempts to deserialize a FileStatus that had been 
> serialized by a 2.5.0 process. The 2.5.0 process would write the ACL, but the 
> 2.3.0 process wouldn't have the code to check the ACL bit and deserialize.

This is a forward compatibility example.  I guess we could not support it in 
any class implementing Writable since it was not designed to support forward 
compatibility.  We may either make the change to 3.0 only or add another class, 
say FileStatusWithACL.

> Other than the couple FsShell acl commands, and distcp -pa, is there a common 
> use case for a client to know/care about acls? If not, increasing the rpc 
> response size for a few rare use cases probably isn't worth it?

For the RPC response size, we may add a parameter to indicate whether ACL 
should be returned (or we may add a new method, say getFileStatusWithACL.)

> I'm aware of no other use cases for visibility of the ACLs right now. ...

Perhaps, this is the most important question -- Is it useful to add ACL to 
FileStatus, or adding a new method and/or a FileStatus subclass?  It seems only 
help distcp -pa.  It may not be significant enough and so we may want to defer 
implementing it.

> WebHdfs ACL compatibility is broken
> -----------------------------------
>
>                 Key: HDFS-6326
>                 URL: https://issues.apache.org/jira/browse/HDFS-6326
>             Project: Hadoop HDFS
>          Issue Type: Bug
>          Components: webhdfs
>    Affects Versions: 3.0.0, 2.4.0
>            Reporter: Daryn Sharp
>            Assignee: Chris Nauroth
>            Priority: Blocker
>             Fix For: 3.0.0, 2.4.1
>
>         Attachments: HDFS-6326-branch-2.4.patch, HDFS-6326.1.patch, 
> HDFS-6326.2.patch, HDFS-6326.3.patch, HDFS-6326.4.patch, HDFS-6326.5.patch, 
> HDFS-6326.6.patch, aclfsperm.example
>
>
> 2.4 ACL support is completely incompatible with <2.4 webhdfs servers.  The NN 
> throws an {{IllegalArgumentException}} exception.
> {code}
> hadoop fs -ls webhdfs://nn/
> Found 21 items
> ls: Invalid value for webhdfs parameter "op": No enum constant 
> org.apache.hadoop.hdfs.web.resources.GetOpParam.Op.GETACLSTATUS
> [... 20 more times...]
> {code}



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

Reply via email to