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

Doron Cohen commented on LUCENE-1310:
-------------------------------------

You're right! And thanks for cacthing this!
NPE is possible and I see it too with the new test. 
I think the suggested new fix is likely to miss additional matches in the same 
doc. 
I'll later post a test that shows this.

(As a side comment, its useful to post patches named "LUCENE-NNN.patch" where 
NNN is the issue number.
This way JIRA shows which fix is the most recent very clearly, and also, being 
a complete patch, everyone can easily apply the entire patch.
More on this in the Wiki under HowToContribute.)

> Phrase query with term repeated 3 times requires more slop than expected
> ------------------------------------------------------------------------
>
>                 Key: LUCENE-1310
>                 URL: https://issues.apache.org/jira/browse/LUCENE-1310
>             Project: Lucene - Java
>          Issue Type: Bug
>          Components: Search
>    Affects Versions: 2.3.1, 2.3.2
>            Reporter: Grant Glouser
>            Assignee: Doron Cohen
>         Attachments: LUCENE-1310.1.patch, LUCENE-1310.patch, 
> TestSloppyPhraseQuery.java
>
>
> Consider a document with the text "A A A".
> The phrase query "A A A" (exact match) succeeds.
> The query "A A A"~1 (same document and query, just increasing the slop value 
> by one) fails.
> "A A A"~2 succeeds again.
> If the exact match succeeds, I wouldn't expect the same query but with more 
> slop to fail.  The fault seems to require some term to be repeated at least 
> three times in the query, but the three occurrences do not need to be 
> adjacent.  I will attach a file that contains a set of JUnit tests that 
> demonstrate what I mean.

-- 
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to