commons-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <>
Subject [jira] [Commented] (COLLECTIONS-575) Synchronized queue wrapper in QueueUtils
Date Mon, 11 Sep 2017 10:38:02 GMT


ASF GitHub Bot commented on COLLECTIONS-575:

Github user kinow commented on the issue:
    Hi @Xaerxess, had a few minutes to look at the pull request tonight. The code looks neat
and simple to understand. Well indented, with tests, and comments. The Javadocs look well
written too.
    Can't thorough review and merge yet. But just wanted to give an initial feedback. As I
am not well familiar with the [collections] code, and not actively working on it, will wait
until someone else has time to properly review and maybe merge it.
    In case it takes too long, feel free to mention my user name and I will try to program
a couple hours to read through the code and try to check if there is anything missing in the
PR, or if we need to get feedback from other via the mailing list.
    Thanks a lot for your contribution. A great first PR :-)

> Synchronized queue wrapper in QueueUtils
> ----------------------------------------
>                 Key: COLLECTIONS-575
>                 URL:
>             Project: Commons Collections
>          Issue Type: New Feature
>          Components: Collection
>    Affects Versions: 4.0
>            Reporter: Guram Savinov
>              Labels: queue, synchronization, wrapping
>             Fix For: 4.x
> There is no synchronized queue wrapper in QueueUtils like in MapUtils or ListUtils for
> My own case: I want to synchronize CircularFifoQueue operations, for this I need synchronized
queue wrapper.

This message was sent by Atlassian JIRA

View raw message