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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
