Rajeev-01 commented on code in PR #14148:
URL: https://github.com/apache/hudi/pull/14148#discussion_r2458737249
##########
hudi-common/src/main/java/org/apache/hudi/common/util/ConfigUtils.java:
##########
@@ -705,6 +706,15 @@ public static TypedProperties fetchConfigs(
}
}
+ public static boolean isPropertiesInvalid(TypedProperties props) {
+ // If checkSum is present we need to validate
+ boolean invalidChecksum = props.containsKey(TABLE_CHECKSUM.key()) &&
!HoodieTableConfig.validateChecksum(props);
+ // For older versions if checksum is not present, table name needs to be
present to generate the checksum
+ boolean missingRequiredProps = !props.containsKey(TABLE_CHECKSUM.key()) &&
!props.containsKey(NAME.key());
Review Comment:
Agree better for readability.
Done
--
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]
