Integrated: 8268595: java/io/Serializable/serialFilter/GlobalFilterTest.java#id1 failed in timeout

[Jaikiran Pai]

On Tue, 19 Oct 2021 13:00:01 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

> The `GlobalFilterTest` has to 2 `@test` tags. One of them has failed with a timeout as noted in https://bugs.openjdk.java.net/browse/JDK-8268595. The timeout seems to have happened even after the tests had already completed successfully. Like I note in the JBS comments of that issue, I suspect it might have to do with the "-XX:StartFlightRecording:name=DeserializationEvent,dumponexit=true" usage in this test action.
> 
> The commit in this PR removes that second `@test` altogether because (correct me if I'm wrong) from what I understand, this test never enables the DeserializationEvent which means there is no JFR events being captured for deserialization in this test, nor does the test do any JFR events related testing. So, I think this second `@test` is virtually a no-op when it comes to the JFR testing. There's a separate `TestDeserializationEvent` which has a comprehensive testing of the DeserializationEvent.

This pull request has now been integrated.

Changeset: f143d2a8
Author:    Jaikiran Pai <jpai at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/f143d2a88e1972cdce9eb6f61c2eb9754cb89251
Stats:     12 lines in 1 file changed: 0 ins; 12 del; 0 mod

8268595: java/io/Serializable/serialFilter/GlobalFilterTest.java#id1 failed in timeout

Reviewed-by: chegar, rriggs

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

PR: https://git.openjdk.java.net/jdk/pull/6008