[JDK 14] RFR 8232195: Enable BigInteger tests: DivisionOverflow, SymmetricRangeTests and StringConstructorOverflow

Brian Burkhalter brian.burkhalter at oracle.com
Tue Oct 15 15:33:42 UTC 2019


Hi Amy,

This looks OK. It looks like DivisionOverflow was already being run however.

Also, the navigation in your webrev is broken: is your script current?

Thanks,

Brian

> On Oct 14, 2019, at 10:05 PM, Amy Lu <amy.lu at oracle.com> wrote:
> 
> test/jdk/java/math/BigInteger/DivisionOverflow.java
> test/jdk/java/math/BigInteger/StringConstructorOverflow.java
> test/jdk/java/math/BigInteger/SymmetricRangeTests.java
> 
> These tests require huge memory and need to run with -Xmx8g
> They are skipped or tagged with @ignore and not actually run in standard testing by jtreg.
> 
> Please review the patch to enable them by:
> * Adding @requires os.maxMemory > 8g
> * Put them into exclusiveAccess.dirs (so they won't run concurrently) to reduce the risk of impact other test.
> 
> Tested on all platforms with small (<8g) or large memory machines.
> 
> bug: https://bugs.openjdk.java.net/browse/JDK-8232195 <https://bugs.openjdk.java.net/browse/JDK-8232195>
> webrev: http://cr.openjdk.java.net/~amlu/8232195/webrev.00/ <http://cr.openjdk.java.net/~amlu/8232195/webrev.00/>


More information about the core-libs-dev mailing list