the-other-tim-brown opened a new pull request, #6619: URL: https://github.com/apache/hudi/pull/6619
### Change Logs - Removes a confusing method, `getReporter()` in the abstract class MetricsReporter since we want to be calling `stop` the MetricsReporter instances to make sure they are cleaned up properly - Updates Metrics.java to call `stop` method - Updates implementations of MetricsReporter to no longer implement getReporter method - Fixes import order in classes that were touched as part of this PR ### Impact The API for the `MetricsReporter` is changing due to the removal of the `getReporter` method **Risk level: low** ### 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]
