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