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

Fix IndexOutOfBoundsException in Jetty 9, 10, 11, 12 InstrumentedHandler #3912

Merged
merged 1 commit into from Jan 19, 2024

Conversation

joschi
Copy link
Member

@joschi joschi commented Jan 19, 2024

Fixes #3911
Refs #3133
Refs #3174
Refs #3175

@joschi joschi added the bug label Jan 19, 2024
@joschi joschi added this to the 4.2.24 milestone Jan 19, 2024
@joschi joschi self-assigned this Jan 19, 2024
@joschi joschi requested review from a team as code owners January 19, 2024 22:03
Copy link

Qodana Community for JVM

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

Copy link

sonarcloud bot commented Jan 19, 2024

Quality Gate Failed Quality Gate failed

Failed conditions

75.0% Coverage on New Code (required ≥ 80%)
36.2% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@joschi joschi merged commit 7a7f124 into release/4.2.x Jan 19, 2024
8 of 9 checks passed
@joschi joschi deleted the issue-3911 branch January 19, 2024 22:44
dongjoon-hyun pushed a commit to apache/spark that referenced this pull request Jan 28, 2024
### What changes were proposed in this pull request?
The pr aims to upgrade dropwizard metrics from `4.2.21` to `4.2.25`.

### Why are the changes needed?
The last update occurred 3 months ago.

- The new version bringes some bug fixes:
  Fix IndexOutOfBoundsException in Jetty 9, 10, 11, 12 InstrumentedHandler dropwizard/metrics#3912

- The full version release notes:
  https://github.com/dropwizard/metrics/releases/tag/v4.2.25
  https://github.com/dropwizard/metrics/releases/tag/v4.2.24
  https://github.com/dropwizard/metrics/releases/tag/v4.2.23
  https://github.com/dropwizard/metrics/releases/tag/v4.2.22

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

### How was this patch tested?
Pass GA.

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

Closes #44918 from panbingkun/SPARK-46892.

Authored-by: panbingkun <panbingkun@baidu.com>
Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Jetty 12 detailed response metrics bug: IndexOutOfBoundsException
1 participant