[ 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)