Skip to content

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
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.

2 participants