Skip to content

SSR renderer pool leaks on app restart #2325 - #2326

Merged
rymsha merged 1 commit into
masterfrom
ssr-pool-leak-2325
Oct 8, 2026
Merged

rymsha merged 1 commit into
masterfrom
ssr-pool-leak-2325

Conversation

@rymsha

@rymsha rymsha commented Oct 8, 2026

Copy link
Copy Markdown
Contributor

Disable JMX for the renderer pool. Fixes #2325

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 8, 2026 07:45
@codecov

codecov Bot commented Oct 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 22.85%. Comparing base (9082045) to head (88ced24).

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2326   +/-   ##
=======================================
  Coverage   22.85%   22.85%           
=======================================
  Files           1        1           
  Lines          35       35           
=======================================
  Hits            8        8           
  Misses         27       27           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The focused configuration change directly addresses the reported retention path without affecting pool behavior.

0 open findings

What changed in this PR

Disables JMX registration for SSR renderer pools, preventing old application classloaders and Graal engines from being retained after restarts.

Changes:

  • Disables JMX in the renderer pool configuration.
File Description
ServerSideRenderer.java Prevents renderer pools from registering as JMX MBeans.

🧠 Review effort: Balanced


Give feedback about Copilot approvals in this survey to enter a drawing for a $150 gift card.

@rymsha
rymsha merged commit e5578d2 into master Oct 8, 2026
6 checks passed
@rymsha
rymsha deleted the ssr-pool-leak-2325 branch October 8, 2026 07:49
rymsha added a commit that referenced this pull request Oct 8, 2026
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
(cherry picked from commit e5578d2)
rymsha added a commit that referenced this pull request Oct 8, 2026
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
(cherry picked from commit e5578d2)
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.

SSR renderer pool leaks on app restart

3 participants