prashantwason commented on code in PR #8758:
URL: https://github.com/apache/hudi/pull/8758#discussion_r1228718306
##########
hudi-client/hudi-client-common/src/main/java/org/apache/hudi/metadata/HoodieMetadataWriteUtils.java:
##########
@@ -170,6 +171,17 @@ public static HoodieWriteConfig createMetadataWriteConfig(
throw new HoodieMetadataException("Unsupported Metrics Reporter type
" + writeConfig.getMetricsReporterType());
}
}
- return builder.build();
+
+ HoodieWriteConfig metadataWriteConfig = builder.build();
+ // Inline compaction and auto clean is required as we do not expose this
table outside
+ ValidationUtils.checkArgument(!metadataWriteConfig.isAutoClean(),
"Cleaning is controlled internally for Metadata table.");
Review Comment:
These validations are for invariants without which the MDT will not work in
current design. So having these checks make it better as someone may change the
configs without knowing the design fully.
--
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]
