lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "David Smiley (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (SOLR-11976) TokenizerChain is overwriting, not chaining TokenFilters in normalize()
Date Fri, 23 Feb 2018 04:14:00 GMT

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

David Smiley commented on SOLR-11976:
-------------------------------------

+1 for replacing TokenizerChain with CustomAnalyzer.  At a glance it seems the right choice.

> TokenizerChain is overwriting, not chaining TokenFilters in normalize()
> -----------------------------------------------------------------------
>
>                 Key: SOLR-11976
>                 URL: https://issues.apache.org/jira/browse/SOLR-11976
>             Project: Solr
>          Issue Type: Bug
>      Security Level: Public(Default Security Level. Issues are Public) 
>          Components: search
>    Affects Versions: master (8.0)
>            Reporter: Tim Allison
>            Priority: Minor
>          Time Spent: 10m
>  Remaining Estimate: 0h
>
> TokenizerChain is overwriting, not chaining tokenfilters in {{normalize}}.  
> This doesn't currently break search because {{normalize}} is not being used at the Solr
level (AFAICT); rather, TextField has its own {{analyzeMultiTerm()}} that duplicates code
from the newer {{normalize}}. 
> Code as is:
> {noformat}
>     TokenStream result = in;
>     for (TokenFilterFactory filter : filters) {
>       if (filter instanceof MultiTermAwareComponent) {
>         filter = (TokenFilterFactory) ((MultiTermAwareComponent) filter).getMultiTermComponent();
>         result = filter.create(in);
>       }
>     }
> {noformat}
> The fix is simple:
> {noformat}
> -        result = filter.create(in);
> +        result = filter.create(result);
> {noformat}



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

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


Mime
View raw message