RFR: 8301341: LinkedTransferQueue does not respect timeout for poll() [v6]

Doug Lea dl at openjdk.org
Thu Jul 20 10:26:09 UTC 2023


On Mon, 17 Jul 2023 14:45:57 GMT, Andrey Turbanov <aturbanov at openjdk.org> wrote:

>> Doug Lea 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 seven additional commits since the last revision:
>> 
>>  - Merge branch 'openjdk:master' into JDK-8301341
>>  - Simplify contention handling; fix test
>>  - Fix inverted test assert; improve internal documentation; simplify code
>>  - Merge branch 'openjdk:master' into JDK-8301341
>>  - Overhaul LTQ and SQ to use common blocking and matching mechanics
>>  - Merge branch 'openjdk:master' into JDK-8301341
>>  - Use Thread.isVirtual to distinguish spin vs immediate block cases
>
> src/java.base/share/classes/java/util/concurrent/SynchronousQueue.java line 232:
> 
>> 230:      * The transferer. (See below about serialization.)
>> 231:      */
>> 232:     private transient final Transferer<E> transferer;
> 
> let's use blessed modifiers order
> Suggestion:
> 
>     private final transient Transferer<E> transferer;

Thanks. Done

> src/java.base/share/classes/java/util/concurrent/SynchronousQueue.java line 234:
> 
>> 232:     private transient final Transferer<E> transferer;
>> 233: 
>> 234:     private transient final boolean fair;
> 
> nit
> Suggestion:
> 
>     private final transient boolean fair;

Thanks. Done.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/14317#discussion_r1269256519
PR Review Comment: https://git.openjdk.org/jdk/pull/14317#discussion_r1269257030


More information about the core-libs-dev mailing list