[12] RFR of JDK-8210353: Move java/util/Arrays/TimSortStackSize2.java back to tier1

[Amy Lu]

Thank you for reviewing!

Change now pushed.

Thanks,
Amy

On 2018/10/23 1:24 PM, Weijun Wang wrote:
> +1.
>
> Thanks
> Max
>
>> On Oct 11, 2018, at 2:27 PM, Remi Forax <forax at univ-mlv.fr> wrote:
>>
>> Looks good !
>>
>> Rémi
>>
>> ----- Mail original -----
>>> De: "Amy Lu" <amy.lu at oracle.com>
>>> À: "core-libs-dev" <core-libs-dev at openjdk.java.net>
>>> Envoyé: Jeudi 11 Octobre 2018 08:13:44
>>> Objet: Re: [12] RFR of JDK-8210353: Move java/util/Arrays/TimSortStackSize2.java back to tier1
>>> Ping ~
>>>
>>> It's no longer running this test in a separate job as test OOM issue
>>> (JDK-8199265) fixed.
>>>
>>> Thanks,
>>> Amy
>>>
>>> On 2018/9/4 5:46 PM, Amy Lu wrote:
>>>> test/jdk/java/util/Arrays/TimSortStackSize2.java
>>>>
>>>> This test was demoted to tier2 due to JDK-8199265 (test fails with
>>>> OOM). This issue has been fixed (in jdk-11+20) and test has been run
>>>> (in tier2) without failure happening after that.
>>>>
>>>> Let's move it back to tier1.
>>>>
>>>> bug: https://bugs.openjdk.java.net/browse/JDK-8210353
>>>> webrev: http://cr.openjdk.java.net/~amlu/8210353/webrev.00/
>>>>
>>>> Thanks,
>>>> Amy
>>>>
>>>> diff --git a/test/jdk/TEST.groups b/test/jdk/TEST.groups
>>>> --- a/test/jdk/TEST.groups
>>>> +++ b/test/jdk/TEST.groups
>>>> @@ -35,8 +35,7 @@
>>>>       :jdk_lang
>>>>
>>>>   tier1_part2 = \
>>>> -    :jdk_util \
>>>> -    -java/util/Arrays/TimSortStackSize2.java
>>>> +    :jdk_util
>>>>
>>>>   tier1_part3 = \
>>>>       :jdk_math \
>>>> @@ -67,9 +66,7 @@
>>>>       -sun/nio/cs/ISO8859x.java \
>>>>       :jdk_other \
>>>>       :jdk_text \
>>>> -    :jdk_time \
>>>> -    java/util/Arrays/TimSortStackSize2.java
>>>> -
>>>> +    :jdk_time
>>>>
>>>>   tier2_part3 = \
>>>>       :jdk_net
>>>> diff --git a/test/jdk/java/util/Arrays/TimSortStackSize2.java
>>>> b/test/jdk/java/util/Arrays/TimSortStackSize2.java
>>>> --- a/test/jdk/java/util/Arrays/TimSortStackSize2.java
>>>> +++ b/test/jdk/java/util/Arrays/TimSortStackSize2.java
>>>> @@ -25,7 +25,6 @@
>>>>    * @test
>>>>    * @bug 8072909
>>>>    * @summary Test TimSort stack size on big arrays
>>>> - * @key intermittent
>>>>    * @library /lib/testlibrary /test/lib
>>>>    * @modules java.management
>>>>    *          java.base/jdk.internal