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