On Tue, 9 Sep 2025 11:46:17 GMT, Mark Sheppard <[email protected]> wrote:
>> Johny Jose 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 two additional
>> commits since the last revision:
>>
>> - Merge branch 'openjdk:master' into JDK-8365398
>> - 8365398: TEST_BUG:
>> java/rmi/transport/checkLeaseInfoLeak/CheckLeaseLeak.java failing
>> intermittently
>
> test/jdk/java/rmi/transport/checkLeaseInfoLeak/CheckLeaseLeak.java line 130:
>
>> 128: /* numLeft should be 6 - if 11 there is a problem. */
>> 129: if (numLeft > 6) {
>> 130: TestLibrary.bomb("Too many objects in DGCImpl.leaseTable: "+
>
> This doesn't solve the problem, the structure of the test is problematic
> please c.f. comments in bug
Synchronized the code where DGCLeaseTable is read and verified the test.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/26815#discussion_r2336373029
