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

Fix #4066: shut down executors when IORuntime.global shuts down #4067

Merged
merged 3 commits into from
Apr 26, 2024

Conversation

durban
Copy link
Contributor

@durban durban commented Apr 25, 2024

Fixes #4066. Related to #4055.

Copy link
Member

@armanbilge armanbilge left a comment

Choose a reason for hiding this comment

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

thanks!

Copy link
Member

@djspiewak djspiewak left a comment

Choose a reason for hiding this comment

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

Oh I forgot that we completely suppressed the shutdown. So we kinda did this to ourselves. Nice catch!

"shut down WSTP on fatal error without IOApp" in {
val h = platform(FatalErrorShutsDownRt, List.empty)
h.awaitStatus()
h.stdout() must not(contain("sadness"))
Copy link
Member

Choose a reason for hiding this comment

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

lol

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That wasn't me, I've copied it from another test 😄

@djspiewak djspiewak merged commit 771a51c into typelevel:series/3.5.x Apr 26, 2024
28 of 31 checks passed
@armanbilge armanbilge changed the title Fix #4066: shut down executors when IORuntime.global shuts down Fix #4066: shut down executors when IORuntime.global shuts down Oct 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

When IORuntime.global shuts down, it doesn't shut down the WSTP
3 participants