hudi
hudi copied to clipboard
[HUDI-7391] HoodieMetadataMetrics should use Metrics instance for metrics registry
Change Logs
Currently HoodieMetadataMetrics stores metrics in memory and these metrics are not pushed by the metric reporters. The metric reporters are configured within Metrics instance. List of changes in the PR:
- Metrics related classes have been moved from hudi-client-common to hudi-common.
- HoodieMetadataMetrics now uses Metrics class so that all the reporters can be supported with it.
- Some gaps in configs which are added in HoodieMetadataWriteUtils
- Some metrics related apis and functionality has been moved to HoodieMetricsConfig. The HoodieWriteConfig APIs now delegate to HoodieMetricsConfig for the functionality.
Impact
There are constructors and APIs which were using HoodieWriteConfig earlier. Now they are using HoodieMetricsConfig. HoodieMetricsConfig can be obtained by calling getMetricsConfig() from the HoodieWriteConfig object.
Risk level (write none, low medium or high below)
low
Documentation Update
There are constructors and APIs which were using HoodieWriteConfig earlier. Now they are using HoodieMetricsConfig. HoodieMetricsConfig can be obtained by calling getMetricsConfig() from the HoodieWriteConfig object.
Contributor's checklist
- [ ] Read through contributor's guide
- [ ] Change Logs and Impact were stated clearly
- [ ] Adequate tests were added if applicable
- [ ] CI passed
hey @prashantwason : can you review this patch please
CI report:
- 87b39716e5db65cd75196fc2150476f674e0baab Azure: PENDING
Bot commands
@hudi-bot supports the following commands:@hudi-bot run azurere-run the last Azure build
Azure CI passed: https://dev.azure.com/apache-hudi-ci-org/apache-hudi-ci/_build/results?buildId=23176&view=results