RFR: Refactoring ShenandoahStrDedupStress test to reduce test time

Zhengyu Gu zgu at redhat.com
Tue Jul 17 19:20:41 UTC 2018



On 07/17/2018 01:16 PM, Aleksey Shipilev wrote:
> On 07/17/2018 07:03 PM, Zhengyu Gu wrote:
>> This refactoring reduces ShenandoahStreDedupStress fastdebug test time from ~24 minutes to about 4
>> minutes.
>>
>> Of course, the test is less stress than it used to. It used to run > 50 GC cycles, now down to 6 -
>> 10 cycles for heuristics other than "aggressive", but enough cycles to trigger string deduplications.
>>
>> Webrev: http://cr.openjdk.java.net/~zgu/shenandoah/strdedup_stress/webrev.00/
> 
> Change looks good, except:
> 
>   *) Please break the lines like this, and have the newline after each @run block:
>        https://paste.fedoraproject.org/paste/BeCms3o-YIAspObdeQ01Ow
> 
>   *) Java style for properties is camelCase, so -DtargetStrings=#, etc.

Updated and retested.

Webrev: 
http://cr.openjdk.java.net/~zgu/shenandoah/strdedup_stress/webrev.01/

Thanks,

-Zhengyu

> 
> -Aleksey
> 
> 


More information about the shenandoah-dev mailing list