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

Loading a module after workspace() hangs on 0.6+ #22101

Closed
ararslan opened this issue May 27, 2017 · 3 comments
Closed

Loading a module after workspace() hangs on 0.6+ #22101

ararslan opened this issue May 27, 2017 · 3 comments
Labels
bug Indicates an unexpected problem or unintended behavior modules regression Regression in behavior compared to a previous version
Milestone

Comments

@ararslan
Copy link
Member

I'm seeing this in 0.6-rc2 and current master:

              _
   _       _ _(_)_     |  A fresh approach to technical computing
  (_)     | (_) (_)    |  Documentation: https://docs.julialang.org
   _ _   _| |_  __ _   |  Type "?help" for help.
  | | | | | | |/ _` |  |
  | | |_| | | | (_| |  |  Version 0.7.0-DEV.371 (2017-05-27 13:28 UTC)
 _/ |\__'_|_|_|\__'_|  |  Commit 71d14aeaec* (0 days old master)
|__/                   |  x86_64-apple-darwin16.6.0

julia> using Distributions

julia> workspace()

julia> using Distributions

This hangs seemingly indefinitely using 99.9% CPU. Ctrl+C does not interrupt the process; it must be forcibly killed.

It works on 0.5, it just emits a slew of method overwritten warnings rather than hanging.

@ararslan ararslan added bug Indicates an unexpected problem or unintended behavior modules regression Regression in behavior compared to a previous version labels May 27, 2017
@RalphAS
Copy link

RalphAS commented May 27, 2017

Similar on Linux 0.6.0-rc2, except I was able to get a stacktrace showing the cycle:

Stacktrace:
 [1] _include_from_serialized(::String) at ./loading.jl:157
 [2] _require_from_serialized(::Int64, ::Symbol, ::String, ::Bool) at ./loading.jl:200
 [3] _require_search_from_serialized(::Int64, ::Symbol, ::String, ::Bool) at ./loading.jl:236
 [4] _require(::Symbol) at ./loading.jl:434
 [5] require(::Symbol) at ./loading.jl:398
 [6] _include_from_serialized(::String) at ./loading.jl:157
 [7] _require_from_serialized(::Int64, ::Symbol, ::String, ::Bool) at ./loading.jl:200
 [8] _require_search_from_serialized(::Int64, ::Symbol, ::String, ::Bool) at ./loading.jl:236
 [9] _require(::Symbol) at ./loading.jl:434
 [10] require(::Symbol) at ./loading.jl:398

@vchuravy
Copy link
Member

Should be fixed by #21714?

@ararslan
Copy link
Member Author

Yes indeed. Thanks for the reference.

ararslan added a commit that referenced this issue May 30, 2017
ararslan added a commit that referenced this issue May 31, 2017
ararslan added a commit that referenced this issue May 31, 2017
@JeffBezanson JeffBezanson added this to the 0.6.0 milestone Jun 1, 2017
ararslan added a commit that referenced this issue Jun 1, 2017
@ararslan ararslan closed this as completed Jun 1, 2017
ararslan pushed a commit that referenced this issue Jun 1, 2017
* workspace command: fix for missing incremental support

* Add a test for #22101
tkelman pushed a commit that referenced this issue Jun 3, 2017
* workspace command: fix for missing incremental support

* Add a test for #22101

(cherry picked from commit 5c90d2e)
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 modules regression Regression in behavior compared to a previous version
Projects
None yet
Development

No branches or pull requests

4 participants