Hi Brent,
see my comments inline:
>Четверг, 1 августа 2019, 9:23 +03:00 от Laurent Bourgès
>:
>
>Hi Brent,
>
>Le jeu. 1 août 2019 à 02:32, Brent Christian < brent.christ...@oracle.com > a
>écrit :
>>Thanks, Laurent. I can sponsor this fix, get a RFR thread going for
>>JDK-8226297, keep
Hi Brent,
Le jeu. 1 août 2019 à 02:32, Brent Christian a
écrit :
> Thanks, Laurent. I can sponsor this fix, get a RFR thread going for
> JDK-8226297, keep webrevs updated as the review progresses, etc.
>
Excellent, I will let you and Vladimir work on this review.
FYI I have implemented DPQS
Thanks, Laurent. I can sponsor this fix, get a RFR thread going for
JDK-8226297, keep webrevs updated as the review progresses, etc.
However I've uncovered an issue with the fix that should be resolved
before proceeding.
Specifically, the new Arrays.COMMON_PARALLELISM static constant causes
Hi all,
I updated the DPQS patch thanks to latest release from Vladimir ([image:
Pièces jointes]2019.07.25):
http://cr.openjdk.java.net/~lbourges/dpqs/dpqs-8226297/dpqs-8226297.1/
"
Updated version of Arrays and DualPivotQuicksort files, the summary of
changes:
1. Corrected minimal threshold
>
>> Is the webrev incomplete? It only contains the changes for
>> DualPivotQuicksort.java, and not for Arrays.java, etc.
>>
>> Thanks,
>> -Brent
>>
>> On 6/18/19 2:37 PM, Vladimir Yaroslavskiy wrote:
>> > Hi team,
>> >
>> > Pleas
ys.java, etc.
>
> Thanks,
> -Brent
>
> On 6/18/19 2:37 PM, Vladimir Yaroslavskiy wrote:
> > Hi team,
> >
> > Please review the final optimized version of Dual-Pivot Quicksort
> (ver.19.1),
> > see http://cr.openjdk.java.net/~alanb/8226297/0/webrev
> >
Hi,
Is the webrev incomplete? It only contains the changes for
DualPivotQuicksort.java, and not for Arrays.java, etc.
Thanks,
-Brent
On 6/18/19 2:37 PM, Vladimir Yaroslavskiy wrote:
Hi team,
Please review the final optimized version of Dual-Pivot Quicksort (ver.19.1),
see http
Hi team,
Please review the final optimized version of Dual-Pivot Quicksort (ver.19.1),
see http://cr.openjdk.java.net/~alanb/8226297/0/webrev
(link to Jira task https://bugs.openjdk.java.net/browse/JDK-8226297)
The new version was published here more than one year ago (on Jan 19, 2018).
Since