performance regression with `@parallel` #12794
Closed
amitmurthy opened this Issue on Aug 25, 2015 · 0 comments
Projects
None yet
Labels
benchmark
parallel
performance
regression
Milestone
 
0.4.0
Assignees

@JeffBezanson JeffBezanson
4 participants
@amitmurthy
@ViralBShah
@JeffBezanson
@jrevels
Notifications

You’re not receiving notifications from this thread.
@amitmurthy
The Julia Language member
amitmurthy commented on Aug 25, 2015

@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 parallel regression labels on Aug 25, 2015
@ViralBShah ViralBShah added this to the 0.4.0 milestone on Aug 25, 2015
@JeffBezanson JeffBezanson self-assigned this on Aug 25, 2015
@JeffBezanson JeffBezanson added a commit that closed this issue on Aug 26, 2015
	@JeffBezanson 	fix #12794, perf regression in parallel macro 			d4d8a2e
@JeffBezanson JeffBezanson closed this in d4d8a2e on Aug 26, 2015
@jrevels jrevels referenced this issue on Nov 6, 2015
Closed
CI Performance Tracking for v0.5 #13893
4 of 4 tasks complete
@jrevels jrevels added the benchmark label on Nov 12, 2015