RFR: 8336240: Test com/sun/crypto/provider/Cipher/DES/PerformanceTest.java fails with java.lang.ArithmeticException [v4]

Bradford Wetmore wetmore at openjdk.org
Fri Jul 19 20:34:31 UTC 2024


On Fri, 19 Jul 2024 14:04:12 GMT, Fernando Guallini <fguallini at openjdk.org> wrote:

> > LGTM. Have you run test with iterations to ensure stability. This will be a tier1 test so intermittent failures are not acceptable.
> 
> 👍 This is a manual test. In any case it's stable, it was successfully ran hundreds of times.

I see it was previously a `/manual` test, before that was an `@ignore`.   The reason it was moved to `@ignore` in 2003 was:

> Exclude this test from regression tests since it is meant to measure performance and not regression testing; it can be run separately if needed.

Is it slow enough these days that we couldn't remove the `/manual`?   How long is the test taking?

@valeriepeng, any reason to keep/not keep this?

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

PR Comment: https://git.openjdk.org/jdk/pull/20135#issuecomment-2240064200



More information about the security-dev mailing list