Mohit20057 commented on PR #9169: URL: https://github.com/apache/storm/pull/9169#issuecomment-6400253409
@rzo1 Thanks! I have updated the PR accordingly: 1. Updated `docs/metrics_v2.md` to state that `storm.metrics.reporters` was deprecated in 2.3.0 and removed in this release. 2. Removed the extra blank lines in `Config.java`, `Nimbus.java`, and `ConfigUtilsTest.java`. 3. Added the `## Upgrade notes` section to the PR description detailing the removal of `storm.metrics.reporters` and the migration requirement for `blobstore.hdfs.*` to `storm.hdfs.login.*`. 4. Rebased onto latest master, resolved the checkstyle conflicts, and verified `mvn test-compile -pl storm-client,storm-server` passes with 0 Checkstyle violations under JDK 23. -- 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]
