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

Xiaoyu Yao commented on HDFS-12069:
-----------------------------------

Thanks [~cheersyang] for the update. Patch v8 looks good to me. Agree that we 
can fix the cblock usage with a separate ticket. 

3 more minor issues. +1 after that being fixed.

ContainerCache.java
Line 86: NIT: OzoneLevelDBStore should be changed to MetadataStore

OzoneMetadataManager.java
We should also change the LevelDBStore used by the localStorage Handler.

KeyUtils.java
Line 62: LevelDB handle should be MetadataStore handler.


> Ozone: Create a general abstraction for metadata store
> ------------------------------------------------------
>
>                 Key: HDFS-12069
>                 URL: https://issues.apache.org/jira/browse/HDFS-12069
>             Project: Hadoop HDFS
>          Issue Type: Sub-task
>          Components: ozone
>            Reporter: Weiwei Yang
>            Assignee: Weiwei Yang
>         Attachments: HDFS-12069-HDFS-7240.001.patch, 
> HDFS-12069-HDFS-7240.002.patch, HDFS-12069-HDFS-7240.003.patch, 
> HDFS-12069-HDFS-7240.004.patch, HDFS-12069-HDFS-7240.005.patch, 
> HDFS-12069-HDFS-7240.006.patch, HDFS-12069-HDFS-7240.007.patch, 
> HDFS-12069-HDFS-7240.008.patch
>
>
> Create a general abstraction for metadata store so that we can plug other key 
> value store to host ozone metadata. Currently only levelDB is implemented, we 
> want to support RocksDB as it provides more production ready features.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

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