[ 
https://issues.apache.org/jira/browse/SOLR-6680?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

David Smiley updated SOLR-6680:
-------------------------------
    Attachment: SOLR-6680.patch

Updated patch. Added check to detect if, for the current document, if the field 
to highlight is _actually_ multi-valued. If it isn't we can avoid 
TermOffsetsTokenStream, which defeats the optimization in SOLR-6034.

> DefaultSolrHighlighter can sometimes avoid CachingTokenFilter
> -------------------------------------------------------------
>
>                 Key: SOLR-6680
>                 URL: https://issues.apache.org/jira/browse/SOLR-6680
>             Project: Solr
>          Issue Type: Improvement
>          Components: highlighter
>            Reporter: David Smiley
>            Assignee: David Smiley
>             Fix For: 5.0
>
>         Attachments: SOLR-6680.patch, SOLR-6680.patch
>
>
> The DefaultSolrHighlighter (the most accurate one) is a bit over-eager to 
> wrap the token stream in a CachingTokenFilter when 
> hl.usePhraseHighlighter=true.  This wastes memory, and it interferes with 
> other optimizations -- LUCENE-6034.  Furthermore, the internal 
> TermOffsetsTokenStream (used when TermVectors are used with this) wasn't 
> properly delegating reset().



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to