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

Added jersey-micrometer module #5391

Merged
merged 3 commits into from
Sep 22, 2023

Conversation

marcingrzejszczak
Copy link
Contributor

code got ported from Micrometer 1.10.9 and contains metrics and observation listeners

fixes #5371

@jansupol
Copy link
Contributor

Thank you. Unfortunately, it looks like you do not have an ECA, which is required for us to accept PRs.

@marcingrzejszczak marcingrzejszczak force-pushed the jerseyMicrometer branch 2 times, most recently from 551390c to 5933822 Compare August 17, 2023 15:33
code got ported from Micrometer 1.10.9 and contains metrics and observation listeners

related issue: eclipse-ee4j#5371

Signed-off-by: Marcin Grzejszczak <marcin@grzejszczak.pl>
@marcingrzejszczak
Copy link
Contributor Author

Ok now it should be better :)

Signed-off-by: Maxim Nesen <maxim.nesen@oracle.com>
@senivam
Copy link
Contributor

senivam commented Sep 13, 2023

@marcingrzejszczak could you please accept my contribution to your PR at the PR 1 of your fork? Thanks a lot

@marcingrzejszczak
Copy link
Contributor Author

@senivam yup, it's merged!

@jansupol
Copy link
Contributor

@marcingrzejszczak Thank you!

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.

Micrometer Observation support
3 participants