You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@commons.apache.org by "ASF GitHub Bot (JIRA)" <ji...@apache.org> on 2017/09/11 10:38:02 UTC

[jira] [Commented] (COLLECTIONS-575) Synchronized queue wrapper in QueueUtils

    [ https://issues.apache.org/jira/browse/COLLECTIONS-575?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16161045#comment-16161045 ] 

ASF GitHub Bot commented on COLLECTIONS-575:
--------------------------------------------

Github user kinow commented on the issue:

    https://github.com/apache/commons-collections/pull/25
  
    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 :-)
    
    Cheers,
    Bruno


> Synchronized queue wrapper in QueueUtils
> ----------------------------------------
>
>                 Key: COLLECTIONS-575
>                 URL: https://issues.apache.org/jira/browse/COLLECTIONS-575
>             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 example.
> My own case: I want to synchronize CircularFifoQueue operations, for this I need synchronized queue wrapper.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)