Skip to content

[W-21681520] fix(apex): defensive LS shutdown to prevent orphaned processes#7031

Merged
CristiCanizales merged 3 commits into
developfrom
phale/jorje-defensive-shutdown
Mar 20, 2026
Merged

[W-21681520] fix(apex): defensive LS shutdown to prevent orphaned processes#7031
CristiCanizales merged 3 commits into
developfrom
phale/jorje-defensive-shutdown

Conversation

@peternhale

@peternhale peternhale commented Mar 20, 2026

Copy link
Copy Markdown
Contributor
  • Await client.stop() with 3s timeout so LS can gracefully close DB
  • Force-kill child Apex LS processes only when stop() times out
  • Dynamic JDWP port (0) to avoid port-in-use when previous session orphaned
  • Explicit outputChannel dispose, client.dispose(), setClientInstance clear

@W-21681520@ #7010

- Await client.stop() with 3s timeout so LS can gracefully close DB
- Force-kill child Apex LS processes only when stop() times out
- Do not push client to subscriptions; cleanup only in deactivate()
- Dynamic JDWP port (0) to avoid port-in-use when previous session orphaned
- Explicit outputChannel dispose, client.dispose(), setClientInstance clear
@W-21681520@
@peternhale peternhale requested a review from a team as a code owner March 20, 2026 14:46
@peternhale peternhale requested a review from gbockus-sf March 20, 2026 14:46
@peternhale peternhale marked this pull request as draft March 20, 2026 14:46
@peternhale peternhale removed the request for review from gbockus-sf March 20, 2026 14:54
@peternhale peternhale marked this pull request as ready for review March 20, 2026 15:16
@CristiCanizales CristiCanizales self-requested a review March 20, 2026 16:48
@CristiCanizales CristiCanizales merged commit 5398355 into develop Mar 20, 2026
62 of 71 checks passed
@CristiCanizales CristiCanizales deleted the phale/jorje-defensive-shutdown branch March 20, 2026 18:58
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.

2 participants