[GitHub] [ozone] sadanand48 commented on a change in pull request #2781: HDDS-5906. Fix DBScanner to support Datanode DB

[GitBox <gi...@apache.org>]

sadanand48 commented on a change in pull request #2781:
URL: https://github.com/apache/ozone/pull/2781#discussion_r738054059



##########
File path: hadoop-hdds/container-service/src/main/java/org/apache/hadoop/ozone/container/metadata/DatanodeSchemaOneDBDefinition.java
##########
@@ -91,7 +91,7 @@ protected DatanodeSchemaOneDBDefinition(String dbPath) {
 
   @Override
   public DBColumnFamilyDefinition[] getColumnFamilies() {
-    return new DBColumnFamilyDefinition[] {getBlockDataColumnFamily(),
-        getMetadataColumnFamily(), getDeletedBlocksColumnFamily() };
+    return new DBColumnFamilyDefinition[] {getMetadataColumnFamily(),

Review comment:
       In DBScanner we maintain a 1-1 map for tableName to ColumFamilyDefinition but in case of V1 , we just have a single table DEFAULT mapped to multiple columnFamilies, so whichever put happens last in the map, it will pick that columnFamily, hence the order is changed here so that the defaultTable is mapped to BlockDataColumnFamily




-- 
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: issues-unsubscribe@ozone.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org