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 <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

Reply via email to