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

Add threadpool starvation endpoints #5113

Merged
merged 1 commit into from
Apr 27, 2022

Conversation

noahfalk
Copy link
Member

We needed to demonstrate threadpool starvation for the tutorial I am adding.
dotnet/diagnostics#2907

We needed to demonstrate threadpool starvation for the tutorial I am adding.
dotnet/diagnostics#2907
@noahfalk noahfalk requested a review from a team as a code owner April 27, 2022 00:52
noahfalk added a commit to noahfalk/docs that referenced this pull request Apr 27, 2022
Copy link
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

Thanks for adding these @noahfalk

This LGTM, and I'll merge it now.

One note as you work on the related article: Make sure you include text in the article that repeats the explanations in the comments. Comments in code aren't localized on the docs platform, only the markdown text is translated. Non-English speakers should get all the information they need from the article text.

@BillWagner BillWagner merged commit 4c53282 into dotnet:main Apr 27, 2022
noahfalk added a commit to noahfalk/docs that referenced this pull request Apr 28, 2022
noahfalk added a commit to noahfalk/docs that referenced this pull request Apr 28, 2022
noahfalk added a commit to dotnet/docs that referenced this pull request Apr 29, 2022
* Adding a threadpool starvation tutorial

dotnet/diagnostics#2907

This depends on the updated sample code in dotnet/samples#5113

* Apply suggestions from code review

Co-authored-by: David Pine <[email protected]>
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