ChenSammi commented on code in PR #5661:
URL: https://github.com/apache/ozone/pull/5661#discussion_r1446867520
##########
hadoop-hdds/common/src/main/java/org/apache/hadoop/hdds/upgrade/HDDSLayoutFeature.java:
##########
@@ -41,7 +41,9 @@ public enum HDDSLayoutFeature implements LayoutFeature {
WEBUI_PORTS_IN_DATANODEDETAILS(6, "Adding HTTP and HTTPS ports " +
"to DatanodeDetails."),
HADOOP_PRC_PORTS_IN_DATANODEDETAILS(7, "Adding Hadoop RPC ports " +
- "to DatanodeDetails.");
+ "to DatanodeDetails."),
+ LAST_CHUNK_TABLE(8, "Datanode RocksDB Schema Version 3 has an extra table " +
Review Comment:
1. How about change the feature name to HBASE_SUPPORT? For file recovery, a
new table finalize_blocks is also introduced. So for HBASE, as of now, two new
tables are introduced for it. So if use the name HBASE_SUPPORT, we can cover
these two new tables, and also the potential new changes in later.
2. We should support both Schema V3 and Schema V2. For an existing Ozone
cluster with older version, it's likely there will be open Schema V2 containers
there.
--
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]
