[ 
https://issues.apache.org/jira/browse/LUCENE-2277?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Robert Muir updated LUCENE-2277:
--------------------------------

    Attachment: LUCENE-2277.patch

attached is Frank's patch, with a testcase that fails with
ConcurrentModificationException as he describes, and passes
with the patch.

will commit soon.

> QueryNodeImpl throws ConcurrentModificationException on add(List<QueryNode>)
> ----------------------------------------------------------------------------
>
>                 Key: LUCENE-2277
>                 URL: https://issues.apache.org/jira/browse/LUCENE-2277
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: contrib/*
>    Affects Versions: 3.0
>         Environment: all
>            Reporter: Frank Wesemann
>            Assignee: Robert Muir
>            Priority: Critical
>             Fix For: 3.1
>
>         Attachments: addChildren.patch, LUCENE-2277.patch
>
>
> on adding a List of children to a QueryNodeImplemention a 
> ConcurrentModificationException is thrown.
> This is due to the fact that QueryNodeImpl instead of iteration over the 
> supplied list, iterates over its internal clauses List.
> Patch:
> Index: QueryNodeImpl.java
> ===================================================================
> --- QueryNodeImpl.java    (revision 911642)
> +++ QueryNodeImpl.java    (working copy)
> @@ -74,7 +74,7 @@
>            
> .getLocalizedMessage(QueryParserMessages.NODE_ACTION_NOT_SUPPORTED));
>      }
>  
> -    for (QueryNode child : getChildren()) {
> +    for (QueryNode child : children) {
>        add(child);
>      }
>  

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


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

Reply via email to