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

Yonik Seeley commented on LUCENE-2197:
--------------------------------------

Comment from LUCENE-2094 the other issue:
bq. To prevent provided set from being modified while they are in use in a 
filter the given set is copied and wrapped in an immutable instance of 
chararrayset.

We should really avoid this type of nannyism in Lucene.

> StopFilter should not create a new CharArraySet if the given set is already 
> an instance of CharArraySet
> -------------------------------------------------------------------------------------------------------
>
>                 Key: LUCENE-2197
>                 URL: https://issues.apache.org/jira/browse/LUCENE-2197
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: Analysis
>    Affects Versions: 3.1
>            Reporter: Simon Willnauer
>            Priority: Critical
>             Fix For: 3.1
>
>         Attachments: LUCENE-2197.patch
>
>
> With LUCENE-2094 a new CharArraySet is created no matter what type of set is 
> passed to StopFilter. This does not behave as  documented and could introduce 
> serious performance problems. Yet, according to the javadoc, the instance of 
> CharArraySet should be passed to CharArraySet.copy (which is very fast for 
> CharArraySet instances) instead of "copied" via "new CharArraySet()"

-- 
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