On Sat, 20 May 2023 15:56:35 GMT, Daniel D. Daugherty <dcu...@openjdk.org> 
wrote:

> A trivial fix to increase a timeout:
> [JDK-8307783](https://bugs.openjdk.org/browse/JDK-8307783) 
> runtime/reflect/ReflectOutOfMemoryError.java timed out
> 
> Trivial fixes to ProblemList tests:
> [JDK-8308468](https://bugs.openjdk.org/browse/JDK-8308468) ProblemList 
> containers/docker/TestMemoryAwareness.java on linux-x64
> [JDK-8308470](https://bugs.openjdk.org/browse/JDK-8308470) ProblemList 
> javax/management/remote/mandatory/connection/BrokenConnectionTest.java on 
> linux-all
> [JDK-8308471](https://bugs.openjdk.org/browse/JDK-8308471) ProblemList 
> javax/management/remote/mandatory/loading/MissingClassTest.java on 
> windows-x64 w/ loom
> [JDK-8308472](https://bugs.openjdk.org/browse/JDK-8308472) ProblemList 
> javax/management/remote/mandatory/loading/RMIDownloadTest.java on windows-x64 
> w/ loom
> [JDK-8308473](https://bugs.openjdk.org/browse/JDK-8308473) ProblemList 
> java/lang/instrument/NativeMethodPrefixAgent.java with loom

One small proposal inline.

test/hotspot/jtreg/runtime/reflect/ReflectOutOfMemoryError.java line 28:

> 26:  * @bug 8297977
> 27:  * @summary Test that throwing OOM from reflected method gets 
> InvocationTargetException
> 28:  * @run main/othervm/timeout=150 ReflectOutOfMemoryError

Doesn't it make a sense to exclude this test from tier1, if it might took more 
than 2 minutes to complete?

-------------

Marked as reviewed by lmesnik (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/14070#pullrequestreview-1435423205
PR Review Comment: https://git.openjdk.org/jdk/pull/14070#discussion_r1199632388

Reply via email to