RFR: Refactoring ShenandoahStrDedupStress test to reduce test time

Aleksey Shipilev shade at redhat.com
Tue Jul 17 17:16:46 UTC 2018


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.

-Aleksey




More information about the shenandoah-dev mailing list