[
https://issues.apache.org/jira/browse/HDFS-5739?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13865918#comment-13865918
]
Haohui Mai commented on HDFS-5739:
----------------------------------
The name parts looks good.
Since {AclEntry#permissions} is a enum, from a semantic point of view I would
prefer that it is non nullable. Is it possible to simply ignore the value in
{{removeAclEntries}}?
> ACL RPC must allow null name or null permissions in ACL entries.
> ----------------------------------------------------------------
>
> Key: HDFS-5739
> URL: https://issues.apache.org/jira/browse/HDFS-5739
> Project: Hadoop HDFS
> Issue Type: Bug
> Components: hdfs-client, namenode
> Affects Versions: HDFS ACLs (HDFS-4685)
> Reporter: Chris Nauroth
> Assignee: Chris Nauroth
> Attachments: HDFS-5739.1.patch
>
>
> Currently, the ACL RPC defines ACL entries with required fields for name and
> permissions. These fields actually need to be optional. The name can be
> null to represent unnamed ACL entries, such as the file owner or mask.
> Permissions can be null when passed in an ACL spec to remove ACL entries via
> {{FileSystem#removeAclEntries}}.
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)
