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