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

performance regression with @parallel #12794

Closed
amitmurthy opened this issue Aug 25, 2015 · 0 comments
Closed

performance regression with @parallel #12794

amitmurthy opened this issue Aug 25, 2015 · 0 comments
Assignees
Labels
parallelism Parallel or distributed computation performance Must go faster potential benchmark Could make a good benchmark in BaseBenchmarks regression Regression in behavior compared to a previous version
Milestone

Comments

@amitmurthy
Copy link
Contributor

@JeffBezanson , before commit 30ed04c

  | | |_| | | | (_| |  |  Version 0.4.0-dev+6875 (2015-08-21 16:22 UTC)
 _/ |\__'_|_|_|\__'_|  |  Commit 128797f (3 days old master)
|__/                   |  x86_64-linux-gnu

julia> @time @parallel (+) for i=1:100_000_000; randn(); end;
  0.975937 seconds (172.01 k allocations: 7.561 MB)

julia> @time @parallel (+) for i=1:100_000_000; randn(); end;
  0.850533 seconds (3.55 k allocations: 182.357 KB)

julia> @time @parallel (+) for i=1:100_000_000; randn(); end;
  0.817012 seconds (3.55 k allocations: 182.357 KB)

With 30ed04c

  | | |_| | | | (_| |  |  Version 0.4.0-dev+6876 (2015-08-21 17:38 UTC)
 _/ |\__'_|_|_|\__'_|  |  Commit 30ed04c (3 days old master)
|__/                   |  x86_64-linux-gnu

julia> @time @parallel (+) for i=1:100_000_000; randn(); end;
 10.096709 seconds (300.17 M allocations: 5.968 GB, 3.52% gc time)

julia> @time @parallel (+) for i=1:100_000_000; randn(); end;
 10.475335 seconds (300.00 M allocations: 5.961 GB, 3.76% gc time)

NOTE: This is not an issue when wrapped in a function even with 30ed04c:

julia> function foo(); @time @parallel (+) for i=1:100_000_000; randn(); end; end;

julia> foo()
  0.816561 seconds (3.42 k allocations: 175.723 KB)
5660.182992102461

@amitmurthy amitmurthy added performance Must go faster parallelism Parallel or distributed computation regression Regression in behavior compared to a previous version labels Aug 25, 2015
@ViralBShah ViralBShah added this to the 0.4.0 milestone Aug 25, 2015
@JeffBezanson JeffBezanson self-assigned this Aug 25, 2015
@jrevels jrevels added the potential benchmark Could make a good benchmark in BaseBenchmarks label Nov 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
parallelism Parallel or distributed computation performance Must go faster potential benchmark Could make a good benchmark in BaseBenchmarks regression Regression in behavior compared to a previous version
Projects
None yet
Development

No branches or pull requests

4 participants