Skip to content

jmx move to internal package + deprecate old API#15658

Merged
trask merged 5 commits intoopen-telemetry:mainfrom
SylvainJuge:jmx-deprecate-old-api
Dec 19, 2025
Merged

jmx move to internal package + deprecate old API#15658
trask merged 5 commits intoopen-telemetry:mainfrom
SylvainJuge:jmx-deprecate-old-api

Conversation

@SylvainJuge
Copy link
Copy Markdown
Contributor

  • deprecate addClassPathRules and addCustomRules methods in JmxTelemetryBuilder
  • move non-public classes to an internal package

Fixes #14674

* @deprecated use {@link #addRules(InputStream)} instead
*/
// TODO: deprecate this method after 2.23.0 release in favor of addRules
@Deprecated
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[for reviewer] do we have a way to make sure that we don´t forget to remove those in the future ? Should we create an issue and take care of this in 3.0 ? Those methods have been public and non-deprecated only in two minor releases 2.22.0 and 2.23.0.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can remove these after the next release. Even if you forget to do it it isn't really a big deal.

@SylvainJuge SylvainJuge marked this pull request as ready for review December 16, 2025 10:59
@SylvainJuge SylvainJuge requested a review from a team as a code owner December 16, 2025 10:59
@SylvainJuge SylvainJuge self-assigned this Dec 16, 2025
@trask trask merged commit e30ad97 into open-telemetry:main Dec 19, 2025
85 checks passed
@SylvainJuge SylvainJuge deleted the jmx-deprecate-old-api branch January 5, 2026 17:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

implement "idiomatic" jmx metrics library API

3 participants