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

Revert symuploader to 1.1.156602 #7534

Merged
merged 1 commit into from
Jun 16, 2021

Conversation

hoyosjs
Copy link
Member

@hoyosjs hoyosjs commented Jun 16, 2021

The newer versions of the symbol publisher have serious threading issues and parallel use of enumerators that surface Streams getting used after disposed. This should unblock symbol publishing, but it brings back prior memory pressure issues...

@hoyosjs
Copy link
Member Author

hoyosjs commented Jun 16, 2021

@ViktorHofer @epananth

Copy link
Member

@ViktorHofer ViktorHofer left a comment

Choose a reason for hiding this comment

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

Thanks Juan. Can you please file an issue or update the existing one with the details that you just provided here. Let's make sure that we follow-up on the actual fix immediately.

@mmitche
Copy link
Member

mmitche commented Jun 16, 2021

@mmitche mmitche merged commit bdcc258 into dotnet:main Jun 16, 2021
@mangod9
Copy link
Member

mangod9 commented Jun 16, 2021

assume runtime repo would need a manual update to move to this arcade build?

@mmitche
Copy link
Member

mmitche commented Jun 16, 2021

@mangod9 No, arcade itself just needs to get an update against this new build.

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.

4 participants