[ https://issues.apache.org/jira/browse/SOLR-822?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12642631#action_12642631 ]
Hoss Man commented on SOLR-822: ------------------------------- Koji: 1) the patch is a little hard to read ... there seems to be a ton of unrelated whitespace changes (some in files that don't seem like they need modified for this functionality at all) 2) the motivation of adding a new type of plugin that has direct access to the "stream of characters" in the Reader before the tokenizer gets access to it seems like a great idea, but i'm a little unclear as to the specific reason for some of the new apis: CharReader, CharFilter, CharStream. What value do these add beyond something like... {code} public abstract class ReaderWrapperFactory { public void init(Map<String,String> args) { ... } public Map<String,String> getArgs() { ... } public Reader create(Reader input) { return input; } } {code} ? > CharFilter - normalize characters before tokenizer > -------------------------------------------------- > > Key: SOLR-822 > URL: https://issues.apache.org/jira/browse/SOLR-822 > Project: Solr > Issue Type: New Feature > Components: Analysis > Reporter: Koji Sekiguchi > Priority: Minor > Attachments: character-normalization.JPG, sample_mapping_ja.txt, > SOLR-822.patch, SOLR-822.patch > > > A new plugin which can be placed in front of <tokenizer/>. > {code:xml} > <fieldType name="textCharNorm" class="solr.TextField" > positionIncrementGap="100" > > <analyzer> > <charFilter class="solr.MappingCharFilterFactory" > mapping="mapping_ja.txt" /> > <tokenizer class="solr.MappingCJKTokenizerFactory"/> > <filter class="solr.StopFilterFactory" ignoreCase="true" > words="stopwords.txt"/> > <filter class="solr.LowerCaseFilterFactory"/> > </analyzer> > </fieldType> > {code} > <charFilter/> can be multiple (chained). I'll post a JPEG file to show > character normalization sample soon. > MOTIVATION: > In Japan, there are two types of tokenizers -- N-gram (CJKTokenizer) and > Morphological Analyzer. > When we use morphological analyzer, because the analyzer uses Japanese > dictionary to detect terms, > we need to normalize characters before tokenization. > I'll post a patch soon, too. -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online.