yihua opened a new pull request, #7713: URL: https://github.com/apache/hudi/pull/7713
### Change Logs For a config defined by `ConfigProperty` with no default value set, e.g., `DynamoDbBasedLockConfig.DYNAMODB_LOCK_PARTITION_KEY`, even if the inference function is set with `.withInferFunction()`, the inference function is not invoked to generate the config value. This PR fixes the issue so that such a config can still get the inferred value without default value set. ### Impact Correct the config inference. This fixes these two existing configs: `hoodie.write.lock.dynamodb.partition_key` (`DynamoDbBasedLockConfig.DYNAMODB_LOCK_PARTITION_KEY`) and `hoodie.write.lock.zookeeper.lock_key` (`HoodieLockConfig.ZK_LOCK_KEY`). This also makes the proper inference for `hoodie.write.concurrency.early.conflict.detection.strategy` in #6133. ### Risk level low ### Documentation Update N/A ### Contributor's checklist - [ ] Read through [contributor's guide](https://hudi.apache.org/contribute/how-to-contribute) - [ ] Change Logs and Impact were stated clearly - [ ] Adequate tests were added if applicable - [ ] CI passed -- 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]
