[GitHub] [hudi] yihua commented on a diff in pull request #9546: [HUDI-6397][HUDI-6759] Fixing misc bugs w/ metadata table

["yihua (via GitHub)" <gi...@apache.org>]

yihua commented on code in PR #9546:
URL: https://github.com/apache/hudi/pull/9546#discussion_r1317601879


##########
hudi-common/src/main/java/org/apache/hudi/metadata/HoodieTableMetadataUtil.java:
##########
@@ -153,6 +153,8 @@ static boolean isValidSuffix(String suffix) {
   // This suffix and all after that are used for initialization of the various partitions. The unused suffixes lower than this value
   // are reserved for future operations on the MDT.
   private static final int PARTITION_INITIALIZATION_TIME_SUFFIX = 10; // corresponds to "010";
+  // we have max of 4 partitions (FILES, COL_STATS, BLOOM, RLI)
+  private static final List<String> VALID_PARTITION_INITIALIZATION_TIME_SUFFIXES = Arrays.asList("010","011","012","013");

Review Comment:
   Makes sense.  We should still track this as we may add new MDT partition in the future.



-- 
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: commits-unsubscribe@hudi.apache.org

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