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

Adam Hiatt commented on SOLR-199:
---------------------------------

I'll make those changes. I agree that we don't want bloated base classes.

> NGramTokenizerFactory is refering to constants from, and constructing an 
> instance of, EdgeNGramTokenizer 
Are you saying that this worries you b/c it is referenced in the example schema 
and will thus break without the lucene-analyzers package? I do agree that this 
example should probably be taken out for the time being (at the least).

> N-gram
> ------
>
>                 Key: SOLR-199
>                 URL: https://issues.apache.org/jira/browse/SOLR-199
>             Project: Solr
>          Issue Type: New Feature
>          Components: search
>            Reporter: Adam Hiatt
>            Priority: Trivial
>         Attachments: SOLR-81-ngram.patch
>
>
> This tracks the creation of a patch that adds the n-gram/edge n-gram 
> tokenizing functionality that was initially part of SOLR-81 (spell checking). 
> This was taken out b/c the lucene SpellChecker class removed this dependency. 
> None-the-less, I think this is useful functionality and the addition is 
> trivial. How does everyone feel about such an addition?

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to