geserdugarov opened a new pull request, #10851: URL: https://github.com/apache/hudi/pull/10851
### Change Logs `ConfigProperty.key()` and `ConfigOption.key()` are used for docs generating, and we need to move towards consistent naming of all parameters in Hudi. This MR proposes the first step, which is changing names of Cleaner configuration parameters only. The following table shows proposed changes. | New main key | Previous key in `HoodieCleanConfig` | Previous key in `FlinkOptions` | | ------------- | ------------- | ------------- | | hoodie.cleaner.automatic | hoodie.clean.automatic | | | hoodie.cleaner.async.enabled | hoodie.clean.async | clean.async.enabled | | hoodie.cleaner.multiple.enabled | hoodie.clean.allow.multiple | | | hoodie.cleaner.incremental.enabled | hoodie.cleaner.incremental.mode | | | hoodie.cleaner.parallelism | hoodie.cleaner.parallelism | | | hoodie.cleaner.policy | hoodie.cleaner.policy | clean.policy | | hoodie.cleaner.commits.retained | hoodie.cleaner.commits.retained | clean.retain_commits | | hoodie.cleaner.fileversions.retained | hoodie.cleaner.fileversions.retained | clean.retain_file_versions | | hoodie.cleaner.hours.retained | hoodie.cleaner.hours.retained | clean.retain_hours | | hoodie.cleaner.trigger.strategy | hoodie.clean.trigger.strategy | | | hoodie.cleaner.trigger.max.commits | hoodie.clean.max.commits | | | hoodie.cleaner.delete.bootstrap.base.file | hoodie.cleaner.delete.bootstrap.base.file | | | hoodie.cleaner.failed.writes.policy | hoodie.cleaner.policy.failed.writes | | Currently, we have about 900+ parameters in Hudi, as I mentioned in [HUDI-5738](https://issues.apache.org/jira/browse/HUDI-5738). Note, that my table contains only 899 of them, and missed, for instance, recently added TTL parameters. ### Impact To save backward compatibility, old keys are available, which is done by using `ConfigProperty.withAlternatives("...")` for `HoodieCleanConfig` or `ConfigOption.withFallbackKeys("...")` for `FlinkOptions`. ### Risk level (write none, low medium or high below) Low ### Documentation Update I will prepare corresponding MR with updates to the documentation. ### 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: commits-unsubscr...@hudi.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
