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

Mukul Kumar Singh commented on HDFS-11778:
------------------------------------------

Hi [~nandakumar131], Thanks for the patch. Please find my comments as follows

1) BuckerManagerImpl: 85, 129 please correct the comment,
rather I feel we should move this to MetadataManagerImpl.java where the key is 
being constructed.
2) KeySpaceManager:366, please add a new metric, and update it here
3) KeySpaceManagerProtocol:114, variable name in comments are not correct
4) BucketManagerImpl:97 - the acls during creation should be fetched from the 
KsmBucketInfo.
5) It would be great if we can add a test to TestKeySpaceManager, this will 
help with end to end testing for all the APIs.

> Ozone: KSM: add getBucketInfo
> -----------------------------
>
>                 Key: HDFS-11778
>                 URL: https://issues.apache.org/jira/browse/HDFS-11778
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ozone
>    Affects Versions: HDFS-7240
>            Reporter: Anu Engineer
>            Assignee: Nandakumar
>         Attachments: HDFS-11778-HDFS-7240.000.patch
>
>
> Returns the bucket information if the bucket exists.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: hdfs-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: hdfs-issues-h...@hadoop.apache.org

Reply via email to