Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move Jakarta JMS instrumentation to micrometer-jakarta9 #4201

Merged
merged 1 commit into from Oct 10, 2023

Conversation

shakuzen
Copy link
Member

@shakuzen shakuzen commented Oct 10, 2023

Introduces a new module micrometer-jakarta9 and moves the previously merged JMS instrumentation there. The reason for this is that we tend to try to support a range of versions when possible for our users, and if Jakarta makes breaking changes in the future, we would not be able to support that in a single module (such as micrometer-core, or micrometer-jakarta) unless the new Jakarta artifacts used different coordinates.

The Jakarta JMS instrumentation was introduced in #4007. This moves it, unchanged, to the new module for the above mentioned reason.
This is a portion of the changes that were proposed in #3989.

@shakuzen shakuzen added the enhancement A general enhancement label Oct 10, 2023
@shakuzen shakuzen added this to the 1.12.0-RC1 milestone Oct 10, 2023
Introduces a new module micrometer-jakarta10 and moves the previously merged JMS instrumentation there. The reason for this is that we tend to try to support a range of versions when possible for our users, and if Jakarta makes breaking changes in the future, we would not be able to support that in a single module (such as micrometer-core, or micrometer-jakarta) unless the new Jakarta artifacts used different coordinates.
@shakuzen shakuzen merged commit 6418574 into micrometer-metrics:main Oct 10, 2023
6 checks passed
@shakuzen shakuzen deleted the jakarta10-module branch October 10, 2023 11:55
shakuzen added a commit that referenced this pull request Oct 10, 2023
There is no strong reason to baseline on the higher 10. Setting the baseline to 9 will allow more people to utilize this module.

See gh-4201
@shakuzen
Copy link
Member Author

shakuzen commented Oct 10, 2023

In 9ba2264 I lowered the baseline to Jakarta EE 9. This will allow more people to use the instrumentation and there's no reason I know of to baseline on 10 now.
Edit: missed renaming the package in the previous commit. Fixed now in 6fec15c

@shakuzen shakuzen changed the title Move Jakarta JMS instrumentation to micrometer-jakarta10 Move Jakarta JMS instrumentation to micrometer-jakarta9 Oct 10, 2023
izeye added a commit to izeye/micrometer that referenced this pull request Nov 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement A general enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant