[
https://issues.apache.org/jira/browse/HDFS-8654?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14608913#comment-14608913
]
Arpit Agarwal commented on HDFS-8654:
-------------------------------------
Hi [~anu], my feedback below.
# I think the parsing code can be replaced with Enum.valueOf()
{code}
switch (parts[0].toLowerCase()) {
case OzoneConsts.OZONE_ACL_USER_TYPE:
....
{code}
# Similarly for parsing OzoneAclRights. I think the RW/WR case could be handled
with an additional enum type WR which is the same value as RW.
+1 otherwise.
> OzoneHandler : Add ACL support
> ------------------------------
>
> Key: HDFS-8654
> URL: https://issues.apache.org/jira/browse/HDFS-8654
> Project: Hadoop HDFS
> Issue Type: Sub-task
> Components: ozone
> Reporter: Anu Engineer
> Assignee: Anu Engineer
> Attachments: hdfs-8654-HDFS-7240.001.patch
>
>
> Add ACL support which is needed by Ozone Buckets
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)