[ https://issues.apache.org/jira/browse/LUCENE-4734?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13715444#comment-13715444 ]
Ryan Lauck commented on LUCENE-4734: ------------------------------------ I'm curious what you think about the comment I had in my original patch before calling addIfNoOverlap. It seems wasteful to traverse the phrase candidate list from the beginning every iteration to search for overlaps, and also prevents gathering the raw highlights as mentioned in my previous comment. What do you think about waiting until all phrase candidates are gathered to optionally filter overlaps? > FastVectorHighlighter Overlapping Proximity Queries Do Not Highlight > -------------------------------------------------------------------- > > Key: LUCENE-4734 > URL: https://issues.apache.org/jira/browse/LUCENE-4734 > Project: Lucene - Core > Issue Type: Bug > Components: modules/highlighter > Affects Versions: 4.0, 4.1, 5.0 > Reporter: Ryan Lauck > Assignee: Adrien Grand > Labels: fastvectorhighlighter, highlighter > Fix For: 4.4 > > Attachments: LUCENE-4734-2.patch, lucene-4734.patch, LUCENE-4734.patch > > > If a proximity phrase query overlaps with any other query term it will not be > highlighted. > Example Text: A B C D E F G > Example Queries: > "B E"~10 D > (D will be highlighted instead of "B C D E") > "B E"~10 "C F"~10 > (nothing will be highlighted) > This can be traced to the FieldPhraseList constructor's inner while loop. > From the first example query, the first TermInfo popped off the stack will be > "B". The second TermInfo will be "D" which will not be found in the submap > for "B E"~10 and will trigger a failed match. -- 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: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org