- Notifications
You must be signed in to change notification settings - Fork5.2k
Enable the portable thread pool by default in coreclr#43841
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
janvorli left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
LGTM, thank you!
mangod9 commentedOct 26, 2020
@tommcdon per our discussion last week, want to ensure the eventing issue doesnt cause any debugger issues you were alluding to. |
kouvel commentedOct 27, 2020
Reverted the last commit, back to the original simple commit. CI issues seen so far are being fixed in#43888. |
davidfowl commentedOct 27, 2020
Happy to see this go in! |
kouvel commentedOct 28, 2020
CI didn't seem to pick up some of the changes from the other PR, rebased to latest |
Uh oh!
There was an error while loading.Please reload this page.
Would like to give it some bake time, enable it in some previews, and allow for further experimentation with that implementation. There is some work left to do with events as captured by#38763, it is expected that they would be fixed in the .NET 6 time-frame.