[ 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