Skip to content

Speed up jnlp launch by 58%#287

Open
mgaffigan wants to merge 3 commits intoOpenIntegrationEngine:mainfrom
mgaffigan:feat/avoid-ext-jnlp
Open

Speed up jnlp launch by 58%#287
mgaffigan wants to merge 3 commits intoOpenIntegrationEngine:mainfrom
mgaffigan:feat/avoid-ext-jnlp

Conversation

@mgaffigan
Copy link
Copy Markdown
Contributor

@mgaffigan mgaffigan commented Apr 5, 2026

Speeds launch time by not requiring multiple round-trips for each plugin. Classpath with ballista remained identical. No obvious benefit to the historical design - presumed to be some Java Security Manager pipe dream.

Scenario Before After % change
Cold cache, 100ms latency 8.9s 3.7s ↓58%
Warm cache, 100ms latency 5.8s 0.7s ↓87%

Mildly related: #197

Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

Test Results

  111 files  ±0    214 suites  ±0   6m 58s ⏱️ + 1m 6s
  651 tests  - 3    651 ✅  - 3  0 💤 ±0  0 ❌ ±0 
1 302 runs   - 6  1 302 ✅  - 6  0 💤 ±0  0 ❌ ±0 

Results for commit 77ad485. ± Comparison against base commit be1072d.

This pull request removes 3 tests.
com.mirth.connect.server.servlets.WebStartServletTest ‑ testDoGetExtension
com.mirth.connect.server.servlets.WebStartServletTest ‑ testDoGetExtensionModifiedURL
com.mirth.connect.server.servlets.WebStartServletTest ‑ testDoGetExtensionQueryParams

♻️ This comment has been updated with latest results.

Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan mgaffigan marked this pull request as ready for review April 5, 2026 05:02
@mgaffigan mgaffigan changed the title Embed extension jnlps in main to avoid RTT Speed up client launch by 58% Apr 5, 2026
@mgaffigan mgaffigan changed the title Speed up client launch by 58% Speed up jnlp launch by 58% Apr 5, 2026
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.

1 participant