+1. Thanks Max
> On Oct 11, 2018, at 2:27 PM, Remi Forax <fo...@univ-mlv.fr> wrote: > > Looks good ! > > Rémi > > ----- Mail original ----- >> De: "Amy Lu" <amy...@oracle.com> >> À: "core-libs-dev" <core-libs-dev@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
