only broadcast using/import to nodes which need it

fix #12381 fix #13999
JuliaLang · Jun 28, 2017 · 4187526 · 4187526
1 parent b3a89f3
commit 4187526
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/base/loading.jl b/base/loading.jl
@@ -316,11 +316,16 @@ Windows.
 """
 function require(mod::Symbol)
  _require(mod)
- # After successfully loading notify downstream consumers
+ # After successfully loading, notify downstream consumers
  if toplevel_load[] && myid() == 1 && nprocs() > 1
  # broadcast top-level import/using from node 1 (only)
  @sync for p in procs()
- p == 1 || @async remotecall_fetch(_require, p, mod)
+ p == 1 && continue
+ @async remotecall_wait(p) do
+ if !isbindingresolved(Main, mod) || !isdefined(Main, mod)
+ _require(mod)
+ end
+ end
  end
  end
  for callback in package_callbacks