lokeshj1703 opened a new pull request, #10635: URL: https://github.com/apache/hudi/pull/10635
### 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: 1. Metrics related classes have been moved from hudi-client-common to hudi-common. 2. HoodieMetadataMetrics now uses Metrics class so that all the reporters can be supported with it. 3. Some gaps in configs which are added in HoodieMetadataWriteUtils 4. 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 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 NA ### 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]
