Hi Alan,

Can you point us to the benchmarks that were created? I look back at the 
archive which mentions some benchmark but the sources doesn't look like they 
have been made available on OpenJDK - are they available somewhere else?

(original discussion on DP quicksort implementation: 
http://permalink.gmane.org/gmane.comp.java.openjdk.core-libs.devel/2628)

Sunny

-----Original Message-----
From: Alan Bateman [mailto:alan.bate...@oracle.com] 
Sent: 24 April 2015 17:09
To: Paul Sandoz; Chan, Sunny [Tech]
Cc: 'core-libs-dev@openjdk.java.net'; O'Leary, Kristen [Tech]
Subject: Re: Patch to improve primitives Array.sort()

On 24/04/2015 09:57, Paul Sandoz wrote:
> See here:
>
>    http://cr.openjdk.java.net/~psandoz/tmp/gs/sort/webrev/
>
> Some very quick comments as i have not yet had time to review more closely:
>
> - IANAL so i dunno about the GS copyright in the files.
>
> - The constant MAX_RUN_LENGTH is no longer used so could be removed. But i 
> would like to understand why it's no longer required.
>
> - There is quite a bit of duplication in the tests. AFAICT data sources are 
> all derived from ints that are then converted. The sources could be data 
> providers, so only one test method per data type is required, each data can 
> come with a descriptive string so it shows up in the test reports. The goal 
> here being if another source of data is added (which is derivable) it could 
> be added just once.
>
Also overall with the existing Sorting test should be examined as it tests a 
lot of cases with varying data sizes (and consequentially runs for a long 
time). We should also go back through the archives for all the other benchmarks 
that were created in the move to the dual pivot implementation.

-Alan

Reply via email to