Re: RRFR 8002356: Add ForkJoin common pool and CountedCompleted

2012-12-17 Thread Doug Lea
On 12/13/12 15:16, Mike Duigou wrote: Some notes: Thanks! - The Object padding (pad10 - pad1d) in WorkQueue and ForkJoinPool is sensitive to reference size and compressed OOPS. I was surprised to see Object used rather than long or int. Until we get @Contended, this is black art. Because o

Re: RRFR 8002356: Add ForkJoin common pool and CountedCompleted

2012-12-13 Thread Mike Duigou
Some notes: - The Object padding (pad10 - pad1d) in WorkQueue and ForkJoinPool is sensitive to reference size and compressed OOPS. I was surprised to see Object used rather than long or int. - how is getCommonPoolParallelism() different from commonPool().getParallelism() ? Seems redundant. -

RRFR 8002356: Add ForkJoin common pool and CountedCompleted

2012-12-13 Thread Chris Hegarty
I would like to re-start [1] review discussion of the changes to ForkJoinXXX ( add a default common pool, task tags, and implementation updates ), and the addition of CountedCompleter, as part of part of JEP 155 [2]. These changes are of course coming form Doug and the JSR 166 EG members. I