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

Nandakumar commented on HDFS-11775:
-----------------------------------

Thanks for the review [~vagarychen]. 
bq. seems TestBucketManagerImpl#testCreateBucket only makes sure there is no 
exception. Can we also check if the bucket is correctly created?
For now checking the existence of created bucket in DB using mocked 
MetadataManager is added, and TODO: Use BucketManagerImpl#getBucketInfo to 
verify creation of bucket.

bq.how about adding some wrappers to MetadataManager
Modified MetadataManager to have following methods
* void start();
* void stop() throws IOException;
* Lock readLock();
* Lock writeLock();
* byte[] get(byte[] key);
* void put(byte[] key, byte[] value);
* void batchPut(List<Map.Entry<byte[], byte[]>> list) throws IOException;

Please find the changes in patch 007.

> Ozone: KSM : add createBucket 
> ------------------------------
>
>                 Key: HDFS-11775
>                 URL: https://issues.apache.org/jira/browse/HDFS-11775
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ozone
>    Affects Versions: HDFS-7240
>            Reporter: Anu Engineer
>            Assignee: Nandakumar
>         Attachments: HDFS-11775-HDFS-7240.000.patch, 
> HDFS-11775-HDFS-7240.001.patch, HDFS-11775-HDFS-7240.002.patch, 
> HDFS-11775-HDFS-7240.003.patch, HDFS-11775-HDFS-7240.004.patch, 
> HDFS-11775-HDFS-7240.005.patch, HDFS-11775-HDFS-7240.006.patch, 
> HDFS-11775-HDFS-7240.007.patch
>
>
> Creates a bucket if it does not exist. A precondition to creating a bucket is 
> that a parent volume must exist.



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