On Fri, 24 Apr 2026 10:44:17 GMT, SendaoYan <[email protected]> wrote:
>> Per Minborg has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> Adjust timeout
>
> test/jdk/java/lang/LazyConstant/LazyConstantSafePublicationTest.java line 158:
>
>> 156: long deadline = System.nanoTime() +
>> Utils.adjustTimeout(TimeUnit.MINUTES.toNanos(4));
>> 157: while (t.isAlive()) {
>> 158: t.join(TimeUnit.SECONDS.toMillis(20));
>
> Does this t.join() need use the Utils.adjustTimeout
Right, I think all the times need to use adjustTimeout probably. Including the
one in Producer.
Looking at the current test code, I'm not sure why there is a delay in the
producer. I think what's happening is that the producers is not being
scheduled, and the consumers, which are stuck waiting for it, will wait forever.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/30832#discussion_r3137552057
