RFR (S): 8147075: Rename old GC JTreg tests to the new naming scheme
Dmitry Fazunenko
dmitry.fazunenko at oracle.com
Fri Jan 15 10:49:13 UTC 2016
Hi Jesper,
On 15.01.2016 13:29, Jesper Wilhelmsson wrote:
> Hi Dima,
>
> Is TestBigObj.java not part of any test group?
This test belongs to two groups:
compact1_minimal = \
...
gc/ \
...
hotspot_gc = \
...
gc/
And it's running regularly in nightly (bigobj.java)
> If these tests never triggered any bugs maybe we should remove them
> instead of rename them...
This test do unusual thing: it checks how VM works with objects of huge
size (with many fields).
I can't say it verifies to much, but I don't think we need to remove it.
Thanks for review!
Dima
>
> Anyhow, the changes looks good.
> /Jesper
>
>
> Den 14/1/16 kl. 17:00, skrev Dmitry Fazunenko:
>> Hello,
>>
>> would you please review the following test refactoring (renaming):
>> test/gc/6581734/Test6581734.java -->test/gc/cms/TestMBeanCMS.java
>> test/gc/6845368/bigobj.java --> test/gc/TestBigObj.java
>> test/gc/7072527/TestFullGCCount.java -->
>> test/gc/TestFullGCCount.java
>>
>> CR: https://bugs.openjdk.java.net/browse/JDK-8147075
>> Webrev: http://cr.openjdk.java.net/~dfazunen/8147075/webrev.00/
>>
>> I did also some very minor changes in the code, but the original
>> logic of the
>> tests is preserved.
>> Tested: locally + RBT is in progress.
>>
>> Test renaming is a bad practice and should be done only as an exception.
>> In this case such renaming is quite safe: I searched for the old test
>> names in
>> JIRA and haven't found any open CRs.
>>
>> Thanks,
>> Dima
>>
>>
>>
>>
More information about the hotspot-gc-dev
mailing list