RFR: 8367365: java/math/BigInteger/BigIntegerTest.java failed in jtreg timeout
Daniel Fuchs
dfuchs at openjdk.org
Thu Sep 11 10:55:09 UTC 2025
On Thu, 11 Sep 2025 08:55:02 GMT, Raffaello Giulietti <rgiulietti at openjdk.org> wrote:
> This PR
> * Moves sqrt() and nthRoot() tests to a new subset.
> * Removes a leftover `@run` tag without a timeout.
> * Increases the explicit timeout from 400 to 480.
> * Sorts the jtreg tags according to standard conventions.
LGTM - but I'm not a regular reviewer in this area. I guess the main question is whether Reviewers will be on board with the idea of having this new subset (as it means chances to run it are now 25% instead of 33%...) Personally I think it's a good idea if it makes the time each subset needs to complete more homogeneous.
-------------
Marked as reviewed by dfuchs (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/27211#pullrequestreview-3210628945
More information about the core-libs-dev
mailing list