guihecheng commented on a change in pull request #2794:
URL: https://github.com/apache/ozone/pull/2794#discussion_r817272932



##########
File path: 
hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/keyvalue/helpers/KeyValueContainerUtil.java
##########
@@ -114,10 +114,8 @@ public static void createContainerMetaData(
               "Unrecognized schema version for container: " + schemaVersion);
     }
 
-    ReferenceCountedDB db =
-        new ReferenceCountedDB(store, dbFile.getAbsolutePath());

Review comment:
       @mukul1987 Yes, actually we still have it to be a referenced DB inside 
the following call BlockUtils.addDB().
   After we introduced new DatanodeStore definitions, we will have the logic to 
check whether we want a referenced DB or a RawDB inside BlockUtils.addDB() 
according to the schema version.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to