Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Avoid repeated switches to main thread to check if in command line mode #9531

Merged

Conversation

Erarndt
Copy link
Contributor

@Erarndt Erarndt commented Sep 4, 2024

During solution load, there is a per project switch to the main thread to determine if the language service needs to be initialized. The APIs for this were switched to be safe to call from a background thread and we only need to check once since this can't change once the process is started.

Microsoft Reviewers: Open in CodeFlow

@Erarndt Erarndt requested a review from a team as a code owner September 4, 2024 17:56
@drewnoakes drewnoakes added Tenet-Performance This issue affects the "Performance" tenet. Performance-Scenario-Solution-Open This issue affects solution open performance. Performance-Scenario-General This issue affects performance in general. labels Sep 4, 2024
Copy link
Member

@drewnoakes drewnoakes left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@drewnoakes drewnoakes merged commit 8d5a9b8 into dotnet:main Sep 6, 2024
5 checks passed
@dotnet-policy-service dotnet-policy-service bot added this to the 17.12 milestone Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Performance-Scenario-General This issue affects performance in general. Performance-Scenario-Solution-Open This issue affects solution open performance. Tenet-Performance This issue affects the "Performance" tenet.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants