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

Michael McCandless commented on LUCENE-4511:
--------------------------------------------

Wow, this looks good!  We could also make an outer array w/ one entry (holding 
field name & array of terms I guess) per field, instead of the array of 
booleans marking the transition.

Hmm, but, you are calling terms.iterator once per term in each field?  Can we 
call that only once per field instead?

At some point/density it may be worth union-ing the terms into an A and using 
Terms.intersect ... we've talked about doing that before ... but we should do 
that separately.
                
> TermsFilter might return wrong results if a field is not indexed or not 
> present in the index
> --------------------------------------------------------------------------------------------
>
>                 Key: LUCENE-4511
>                 URL: https://issues.apache.org/jira/browse/LUCENE-4511
>             Project: Lucene - Core
>          Issue Type: Bug
>          Components: modules/other
>    Affects Versions: 4.0, 4.1, 5.0
>            Reporter: Simon Willnauer
>            Assignee: Simon Willnauer
>             Fix For: 4.1, 5.0
>
>         Attachments: LUCENE-4511.patch, LUCENE-4511.patch
>
>
> TermsFilter returns if a term returns null from AIR#terms(term) while it 
> should just continue. I will upload a test & fix shortly

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to