Skip to content

Commit

Permalink
[Distributed] don't sidestep require logic (#26813)
Browse files Browse the repository at this point in the history
  • Loading branch information
vchuravy authored and ararslan committed Aug 2, 2018
1 parent ea5871a commit 3b50b2d
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 1 deletion.
2 changes: 1 addition & 1 deletion stdlib/Distributed/src/Distributed.jl
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ function _require_callback(mod::Base.PkgId)
@sync for p in procs()
p == 1 && continue
@async remotecall_wait(p) do
Base._require(mod)
Base.require(mod)
nothing
end
end
Expand Down
17 changes: 17 additions & 0 deletions stdlib/Distributed/test/distributed_exec.jl
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,23 @@ include(joinpath(Sys.BINDIR, "..", "share", "julia", "test", "testenv.jl"))
addprocs_with_testenv(4)
@test nprocs() == 5

# distributed loading of packages

# setup
@everywhere begin
old_act_proj = Base.ACTIVE_PROJECT[]
pushfirst!(Base.LOAD_PATH, "@")
Base.ACTIVE_PROJECT[] = joinpath(Sys.BINDIR, "..", "share", "julia", "test", "TestPkg")
end

@everywhere using TestPkg
@everywhere using TestPkg

@everywhere begin
Base.ACTIVE_PROJECT[] = old_act_proj
popfirst!(Base.LOAD_PATH)
end

@everywhere using Test, Random, LinearAlgebra

id_me = myid()
Expand Down

1 comment on commit 3b50b2d

@nanosoldier
Copy link
Collaborator

Choose a reason for hiding this comment

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

Executing the daily benchmark build, I will reply here when finished:

@nanosoldier runbenchmarks(ALL, isdaily = true)

Please sign in to comment.