RFR 8014076: Arrays parallel and serial sorting improvements (original) (raw)

Chris Hegarty chris.hegarty at oracle.com
Tue May 7 17:13:25 UTC 2013

On 05/07/2013 05:04 PM, Mike Duigou wrote:

The "currently" MINARRAYSORTGRAN statement bothers me. Can we remove currently?

No problem. That would read...

"When the sub-array length reaches a {@linlplain #MIN_ARRAY_SORT_GRAN minimum granularity}, the sub-array is sorted using the appropriate Arrays.sort method."

I would expect to see currently if the numerical value of MIN_ARRAY_SORT_GRAN was presented. We may change the threshold but we're otherwise committed to the constant name for the threshold.

I really don't care much for MIN_ARRAY_SORT_GRAN. I left it out from the original push, then flip flopped a few times on it. I don't like {@value}, as the field would still need to be public, but not referenced in the docs. I could be persuaded to go either way on it, but it is not worth spending time on.

-Chris.

Mike On May 7 2013, at 07:51 , Chris Hegarty wrote:

Doug has made some updates to the java.util.Arrays sorting code to provide stable sorting. There have also been some changes to the original Parallel Array Sorting ( MINARRAYSORTGRAN is public again ).

Right now a copy of this work is sitting in the lambda repo. This issue proposed to integrate this work into jdk8. http://cr.openjdk.java.net/~chegar/8014076/ver.00/specdiff/java/util/Arrays.html http://cr.openjdk.java.net/~chegar/8014076/ver.00/webrev/ Thanks, -Chris.

More information about the core-libs-dev mailing list