Skip to content

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