Skip to content

Make OJP server virtual-thread execution configurable (default on) and document platform-thread fallback#496

Merged
rrobetti merged 4 commits intomainfrom
copilot/configure-virtual-threads-in-ojp-server
May 8, 2026
Merged

Make OJP server virtual-thread execution configurable (default on) and document platform-thread fallback#496
rrobetti merged 4 commits intomainfrom
copilot/configure-virtual-threads-in-ojp-server

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 8, 2026 •
edited
Loading

  • Re-check current implementation and docs for virtual-thread default behavior
  • Update documentation examples so the default mode is explicitly true (or omit overrides)
  • Keep any explicit conventional-thread examples only where fallback mode is intentionally demonstrated
  • Run targeted checks for touched area
  • Commit, push, and reply to PR comment

rrobetti
rrobetti requested changes May 8, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@rrobetti rrobetti left a comment

Choose a reason for hiding this comment

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

@copilot change defaults to true.

Copy link
Copy Markdown
Contributor Author

Copilot AI commented May 8, 2026

@copilot change defaults to true.

Updated so defaults are true in the docs/examples (and kept explicit disable examples only for fallback guidance) in commit 0fd8428.

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 8, 2026

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Copilot AI requested a review from rrobetti May 8, 2026 08:52
@rrobetti rrobetti marked this pull request as ready for review May 8, 2026 09:38
@rrobetti rrobetti merged commit e4d6628 into main May 8, 2026
53 checks passed
@rrobetti rrobetti deleted the copilot/configure-virtual-threads-in-ojp-server branch May 8, 2026 09:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@rrobetti rrobetti rrobetti approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@rrobetti rrobetti

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@rrobetti