lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Muir (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (LUCENE-3234) Provide limit on phrase analysis in FastVectorHighlighter
Date Thu, 23 Jun 2011 18:01:48 GMT

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

Robert Muir commented on LUCENE-3234:
-------------------------------------

I like this tradeoff Mike, thanks!

should we consider setting some kind of absurd default like 10,000 to really prevent some
pathological cases with huge documents?
We could document in CHANGES.txt that if you want the old behavior, set it to -1 or Integer.MAX_VALUE
(I think we can use this here? offsets are ints?)

> Provide limit on phrase analysis in FastVectorHighlighter
> ---------------------------------------------------------
>
>                 Key: LUCENE-3234
>                 URL: https://issues.apache.org/jira/browse/LUCENE-3234
>             Project: Lucene - Java
>          Issue Type: Improvement
>            Reporter: Mike Sokolov
>         Attachments: LUCENE-3234.patch
>
>
> With larger documents, FVH can spend a lot of time trying to find the best-scoring snippet
as it examines every possible phrase formed from matching terms in the document.  If one is
willing to accept
> less-than-perfect scoring by limiting the number of phrases that are examined, substantial
speedups are possible.  This is analogous to the Highlighter limit on the number of characters
to analyze.
> The patch includes an artifical test case that shows > 1000x speedup.  In a more normal
test environment, with English documents and random queries, I am seeing speedups of around
3-10x when setting phraseLimit=1, which has the effect of selecting the first possible snippet
in the document.  Most of our sites operate in this way (just show the first snippet), so
this would be a big win for us.
> With phraseLimit = -1, you get the existing FVH behavior. At larger values of phraseLimit,
you may not get substantial speedup in the normal case, but you do get the benefit of protection
against blow-up in pathological cases.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@lucene.apache.org
For additional commands, e-mail: dev-help@lucene.apache.org


Mime
View raw message