On Thu, 19 Feb 2026 10:06:40 GMT, Kevin Walls <[email protected]> wrote:
> Test timeout: update to use timeout factor. > > The timeout that fails is short, and failing runs have a test.timeout.factor > of 10 or 16 (Xcomp and low threshold) . Timeout was bumped from 30 to 60 > seconds in a previous change, it's better revert that and apply the timeout. This looks okay. But I've posted a question. test/jdk/com/sun/jmx/remote/NotificationMarshalVersions/TestSerializationMismatch.java line 38: > 36: * @library /test/lib > 37: * @build jdk.test.lib.Utils > 38: * Q: Why this is needed now? Did this test worked before? ------------- Marked as reviewed by sspitsyn (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/29813#pullrequestreview-3828156913 PR Review Comment: https://git.openjdk.org/jdk/pull/29813#discussion_r2829833682
