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

ASF subversion and git services commented on LUCENE-6831:
---------------------------------------------------------

Commit 7362c4ce603d81f276a83070e51bda52c3528bd7 in lucene-solr's branch 
refs/heads/master from Dawid Weiss
[ https://gitbox.apache.org/repos/asf?p=lucene-solr.git;h=7362c4c ]

LUCENE-6831: start removing LinkedList in favor of ArrayList or De/Queues 
(#1969)

I'm committing it in, seems like a trivial thing.

> Review LinkedList usage
> -----------------------
>
>                 Key: LUCENE-6831
>                 URL: https://issues.apache.org/jira/browse/LUCENE-6831
>             Project: Lucene - Core
>          Issue Type: Task
>            Reporter: Dawid Weiss
>            Priority: Trivial
>             Fix For: 6.0
>
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> I quickly scanned the code (out of curiosity) and most of the use cases of 
> LinkedList are as a Queue, in which case indeed an ArrayDeque would be a 
> better choice, especially if the maximum size is known in advance.
> There are also some invalid/ incorrect uses like calling size() on a linked 
> list in {{MultiPhraseQueryNodeBuilder}}, which should be fixed.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org
For additional commands, e-mail: issues-h...@lucene.apache.org

Reply via email to