RFR: JDK-8266431: Dual-Pivot Quicksort improvements (Radix sort) [v7]
iaroslavski
duke at openjdk.java.net
Fri Oct 29 16:09:32 UTC 2021
> 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 with a new target base due to a merge or a rebase. The pull request now contains ten commits:
- JDK-8266431: Dual-Pivot Quicksort improvements (Radix sort)
Merge with external changes
- JDK-8266431: Dual-Pivot Quicksort improvements (Radix sort)
Added more comments
- JDK-8266431: Dual-Pivot Quicksort improvements (Radix sort)
Better naming of methods
- JDK-8266431: Dual-Pivot Quicksort improvements (Radix sort)
Simplified mixed insertion sort
- Merge remote-tracking branch 'upstream/master' into JDK-8266431-Dual-Pivot-Quicksort-improvements-Radix-sort
- JDK-8266431: Dual-Pivot Quicksort improvements (Radix sort)
Update target version
- JDK-8266431: Dual-Pivot Quicksort improvements (Radix sort)
Testing:
- remove @since and @date, otherwise jtreg tag parser fails
- JDK-8266431: Dual-Pivot Quicksort improvements (Radix sort)
Sorting:
- move radix sort out from quicksort partitioning
- rename radixSort to tryRadixSort
- minor javadoc and comment changes
Testing:
- rename radixSort to tryRadixSort in helper
- JDK-8266431: Dual-Pivot Quicksort improvements (Radix sort)
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
-------------
Changes: https://git.openjdk.java.net/jdk/pull/3938/files
Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=3938&range=06
Stats: 1288 lines in 3 files changed: 855 ins; 102 del; 331 mod
Patch: https://git.openjdk.java.net/jdk/pull/3938.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/3938/head:pull/3938
PR: https://git.openjdk.java.net/jdk/pull/3938
More information about the core-libs-dev
mailing list