RFR: 8261483: jdk/dynalink/TypeConverterFactoryMemoryLeakTest.java failed with "AssertionError: Should have GCd a method handle by now" [v4]

[Aleksey Shipilev]

On Sun, 28 Feb 2021 10:28:56 GMT, Attila Szegedi <attila at openjdk.org> wrote:

>> 8261483: jdk/dynalink/TypeConverterFactoryMemoryLeakTest.java failed with "AssertionError: Should have GCd a method handle by now"
>
> Attila Szegedi has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains one additional commit since the last revision:
> 
>   8261483: Eliminate flakiness of the tests by using iteration number limit and explicitly running GC

Good. Minor suggestion follows.

test/jdk/jdk/dynalink/TypeConverterFactoryMemoryLeakTest.java line 25:

> 23: 
> 24: /*
> 25:  * @test id=with_SerialGC

No need to be that explicit ("with_SerialGC"). "id=serial" should be enough.

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

Marked as reviewed by shade (Reviewer).

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