[
https://issues.apache.org/jira/browse/LUCENE-794?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12546181
]
Mark Harwood commented on LUCENE-794:
-------------------------------------
Makes sense to commit it to me.
I want to spend some time reviewing this in more detail once I'm through with
contributing the new web-based version of Luke.
At a quick glance, does the new Junit test in this patch encompass both old and
new Highlighter tests? In which case should we remove the old Junit test if
they overlap?
> Extend contrib Highlighter to properly support phrase queries and span queries
> ------------------------------------------------------------------------------
>
> Key: LUCENE-794
> URL: https://issues.apache.org/jira/browse/LUCENE-794
> Project: Lucene - Java
> Issue Type: Improvement
> Components: Other
> Reporter: Mark Miller
> Priority: Minor
> Attachments: spanhighlighter.patch, spanhighlighter10.patch,
> spanhighlighter11.patch, spanhighlighter12.patch, spanhighlighter2.patch,
> spanhighlighter3.patch, spanhighlighter5.patch, spanhighlighter6.patch,
> spanhighlighter7.patch, spanhighlighter8.patch, spanhighlighter9.patch,
> spanhighlighter_patch_4.zip
>
>
> This patch adds a new Scorer class (SpanQueryScorer) to the Highlighter
> package that scores just like QueryScorer, but scores a 0 for Terms that did
> not cause the Query hit. This gives 'actual' hit highlighting for the range
> of SpanQuerys and PhraseQuery. There is also a new Fragmenter that attempts
> to fragment without breaking up Spans.
> See http://issues.apache.org/jira/browse/LUCENE-403 for some background.
> There is a dependency on MemoryIndex.
--
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]
