On Mon, 14 Mar 2022 19:12:29 GMT, iaroslavski <d...@openjdk.java.net> wrote:

>> Sorting:
>> - adopt radix sort for sequential and parallel sorts on 
>> int/long/float/double arrays (almost random and length > 6K)
>> - fix tryMergeRuns() to better handle case when the last run is a single 
>> element
>> - minor javadoc and comment changes
>> Testing:
>> - add new data inputs in tests for sorting
>> - add min/max/infinity values to float/double testing
>> - add tests for radix sort
> iaroslavski has updated the pull request incrementally with one additional 
> commit since the last revision:
>   JDK-8266431: Dual-Pivot Quicksort improvements (Radix sort)
>   * Improved mixed insertion sort
>   * Optimized insertion sort
>   * Improved merging sort
>   * Optimized soring tests

allocating extra buffers and catching OOME when sorting primitives is rather 
unsatisfactory. you're not giving a reliable option for sorting under low 
memory conditions. IMO at least the single-threaded primitives (ints, longs, 
floats, etc all non-objects) sorting should be frugal when it comes to memory 

just my 2 cents.


PR: https://git.openjdk.java.net/jdk/pull/3938

Reply via email to