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

Move test of stackoverflow with close(::Channel) to stack_overflow.jl #49702

Merged
merged 1 commit into from
May 9, 2023

Conversation

NHDaly
Copy link
Member

@NHDaly NHDaly commented May 9, 2023

Per @vtjnash's comment here:
https://github.com/JuliaLang/julia/pull/49508/files/bca5ac79d04fb2a95f3b9a7b7448fe5b478f950b#r1186161259

the handling of this error is quite bad and sometimes ends up breaking
the process (as I found out today trying to debug something completely
unrelated)

This is a Tests-only PR

Per @vtjnash's comment here:
https://github.com/JuliaLang/julia/pull/49508/files/bca5ac79d04fb2a95f3b9a7b7448fe5b478f950b#r1186161259

> the handling of this error is quite bad and sometimes ends up breaking
> the process (as I found out today trying to debug something completely
> unrelated)

This is a Tests-only PR
@NHDaly
Copy link
Member Author

NHDaly commented May 9, 2023

Whoever reviews this, please feel free to merge immediately; this is a trivial change.

@NHDaly NHDaly requested a review from vtjnash May 9, 2023 17:30
@KristofferC KristofferC merged commit b9b8b38 into master May 9, 2023
@KristofferC KristofferC deleted the nhd-move-test-stackoverflow-channel branch May 9, 2023 20:47
NHDaly added a commit to RelationalAI/julia that referenced this pull request Nov 8, 2023
…JuliaLang#49702)

Per @vtjnash's comment here:
https://github.com/JuliaLang/julia/pull/49508/files/bca5ac79d04fb2a95f3b9a7b7448fe5b478f950b#r1186161259

> the handling of this error is quite bad and sometimes ends up breaking
> the process (as I found out today trying to debug something completely
> unrelated)

This is a Tests-only PR
Drvi pushed a commit to RelationalAI/julia that referenced this pull request Nov 8, 2023
…JuliaLang#49702) (#108)

Per @vtjnash's comment here:
https://github.com/JuliaLang/julia/pull/49508/files/bca5ac79d04fb2a95f3b9a7b7448fe5b478f950b#r1186161259

> the handling of this error is quite bad and sometimes ends up breaking
> the process (as I found out today trying to debug something completely
> unrelated)

This is a Tests-only PR
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