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

Ajay Kumar commented on HDDS-909:
---------------------------------

[~anu] thanks for comments.
{quote}Not sure I understand why we are returning OZONE_URI_DELIMITER in the 
default case.{quote}
Added default option to satisfy style check. Resource type shouldn't be 
something else. New patch throws an Exception for default case. Let me know if 
you think we should do something different. 
{quote}In TestOzoneObjInfo
 .setResType(RESOURCE_TYPE)
SetResourceType(resource) is not very easy to understand, did you intended to 
write the setResType(Volume_Type){quote}
Removed the field, directly using enum now.
{quote}in TestOzoneObjInfo#testGetBucketName - line 69, 70 looks same as 72,73. 
Just want to make sure it is intentional.{quote}
Thanks for catching it, removed.

Fixed the checkstyle issued for length.


> Default implementation for Ozone acls
> -------------------------------------
>
>                 Key: HDDS-909
>                 URL: https://issues.apache.org/jira/browse/HDDS-909
>             Project: Hadoop Distributed Data Store
>          Issue Type: New Feature
>            Reporter: Ajay Kumar
>            Assignee: Ajay Kumar
>            Priority: Major
>         Attachments: HDDS-909.00.patch, HDDS-909.01.patch, HDDS-909.02.patch
>
>
> Default implementation for Ozone acls



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