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

Race condition in include_from_node1 #11093

Closed
simonster opened this issue May 1, 2015 · 2 comments · Fixed by #22588
Closed

Race condition in include_from_node1 #11093

simonster opened this issue May 1, 2015 · 2 comments · Fixed by #22588
Labels
bug Indicates an unexpected problem or unintended behavior parallelism Parallel or distributed computation

Comments

@simonster
Copy link
Member

As I noted in #11092 (comment), include_from_node1 is vulnerable to a race condition when Julia has more than one worker, if another task gets run while it's sleeping and starts loading a file. Not a big deal, but here's an example that triggers it:

$ julia -p1

[...]

julia> function f()
           @async reload("HDF5")
           reload("Distributions")
       end;

julia> f()
ERROR: LoadError: LoadError: could not open file /Users/simon/.julia/v0.4/ArrayViews/src/JLD.jl
 in include at ./boot.jl:250
 in include_from_node1 at ./loading.jl:129
 in include at ./boot.jl:250
 in include_from_node1 at ./loading.jl:129
 in reload_path at ./loading.jl:153
 in _require at ./loading.jl:68
 in require at ./loading.jl:54
 in include at ./boot.jl:250
 in include_from_node1 at ./loading.jl:129
 in reload_path at ./loading.jl:153
 in reload at loading.jl:86
 in f at none:3
while loading /Users/simon/.julia/v0.4/ArrayViews/src/ArrayViews.jl, in expression starting on line 2
while loading /Users/simon/.julia/v0.4/Distributions/src/Distributions.jl, in expression starting on line 3
@simonster simonster added bug Indicates an unexpected problem or unintended behavior parallelism Parallel or distributed computation labels May 1, 2015
@StefanKarpinski StefanKarpinski added this to the 0.6.0 milestone Sep 13, 2016
@tkelman
Copy link
Contributor

tkelman commented Dec 22, 2016

was this fixed by #18504?

@StefanKarpinski
Copy link
Member

StefanKarpinski commented Feb 2, 2017

This is just a bug and doesn't seem to be a new one at that. Not release blocking.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior parallelism Parallel or distributed computation
Projects
None yet
3 participants