Skip to content

Conversation

@sbordet
Copy link
Contributor

@sbordet sbordet commented Dec 29, 2025

No description provided.

@sbordet sbordet requested review from joakime and lorban December 29, 2025 14:12
@sbordet sbordet self-assigned this Dec 29, 2025
@sbordet sbordet moved this to 👀 In review in Jetty 12.1.6 Dec 29, 2025
@sbordet sbordet removed this from Jetty 12.1.6 Dec 29, 2025
@sbordet sbordet moved this to 👀 In review in Jetty 12.0.32 Dec 29, 2025
@sbordet sbordet linked an issue Dec 29, 2025 that may be closed by this pull request
Copy link
Contributor

@lorban lorban left a comment

Choose a reason for hiding this comment

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

LGTM but:

  • why 12.0 instead of 12.1?
  • how does this improve performance?

@sbordet
Copy link
Contributor Author

sbordet commented Jan 5, 2026

There are 2 PRs: one for 12.0.x and one for 12.1.x, see #14261 and #14262.

@lorban lorban changed the title Fixes #14260 - Improve GZIP performance 12.0: Fixes #14260 - Improve GZIP performance Jan 5, 2026
@lorban
Copy link
Contributor

lorban commented Jan 5, 2026

@sbordet ok, just for clarity I renamed the title of this PR to explicitly state that it's about 12.0, as having two PRs against 12.0 and 12.1 both with the exact same title is what confused me.

@sbordet sbordet merged commit 5578dfb into jetty-12.0.x Jan 5, 2026
9 checks passed
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Jetty 12.0.32 Jan 5, 2026
@sbordet sbordet deleted the fix/jetty-12.0.x/14260-improve-gzip-performance branch January 5, 2026 10:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

Improve GZIP performance

4 participants