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

Correction for the Jetty-12 QTP metrics #4181

Merged
merged 7 commits into from
Jun 9, 2024

Conversation

dkaukov
Copy link
Contributor

@dkaukov dkaukov commented May 23, 2024

This uses Jetty-12 recommended way to calculate thread pool utilization. Taking into account number of threads that are leased to internal components, and therefore cannot be used to execute transient jobs.

This uses Jetty-12 recommended way to calculate thread pool utilization. Taking into account number of threads that are leased to internal components, and therefore cannot be used to execute transient jobs.
@dkaukov dkaukov requested review from a team as code owners May 23, 2024 02:23
@joschi joschi added this to the 4.2.26 milestone Jun 9, 2024
@joschi joschi assigned joschi and dkaukov and unassigned joschi Jun 9, 2024
Copy link
Member

@joschi joschi left a comment

Choose a reason for hiding this comment

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

@dkaukov Awesome, thanks for your contribution!

@joschi joschi merged commit 9e0bedb into dropwizard:release/4.2.x Jun 9, 2024
5 checks passed
joschi pushed a commit that referenced this pull request Jun 9, 2024
joschi added a commit that referenced this pull request Jun 9, 2024
Refs #4181
(cherry picked from commit 9e0bedb)

Co-authored-by: Dmitry Kaukov <dkaukov@opentable.com>
yaooqinn pushed a commit to apache/spark that referenced this pull request Jun 12, 2024
### What changes were proposed in this pull request?

Upgrade dropwizard metrics to 4.2.26.

### Why are the changes needed?

There are some bug fixes as belows:

- Correction for the Jetty-12 QTP metrics by dkaukov in dropwizard/metrics#4181

- Fix metrics for InstrumentedEE10Handler by zUniQueX in dropwizard/metrics#3928

The full release notes:
https://github.com/dropwizard/metrics/releases/tag/v4.2.26

### Does this PR introduce _any_ user-facing change?

No.

### How was this patch tested?

Passed GA.

### Was this patch authored or co-authored using generative AI tooling?

No.

Closes #46932 from wayneguow/codahale.

Authored-by: Wei Guo <guow93@gmail.com>
Signed-off-by: Kent Yao <yao@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants