[ https://issues.apache.org/jira/browse/LUCENE-1359?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Mark Miller updated LUCENE-1359: -------------------------------- Priority: Minor (was: Major) Depends on how you read things - it must be able to handle null for backward compatibility - but contrib analyzers have no explicit back compat requirement - so not supporting such a thing, while inconvenient, is not necessarily wrong. > FrenchAnalyzer's tokenStream method does not honour the contract of Analyzer > ---------------------------------------------------------------------------- > > Key: LUCENE-1359 > URL: https://issues.apache.org/jira/browse/LUCENE-1359 > Project: Lucene - Java > Issue Type: Bug > Components: Analysis > Affects Versions: 2.2 > Reporter: Andrew Lynch > Priority: Minor > > In {{Analyzer}} : > {code} > /** Creates a TokenStream which tokenizes all the text in the provided > Reader. Default implementation forwards to tokenStream(Reader) for > compatibility with older version. Override to allow Analyzer to choose > strategy based on document and/or field. Must be able to handle null > field name for backward compatibility. */ > public abstract TokenStream tokenStream(String fieldName, Reader reader); > {code} > and in {{FrenchAnalyzer}} > {code} > public final TokenStream tokenStream(String fieldName, Reader reader) { > if (fieldName == null) throw new IllegalArgumentException("fieldName must > not be null"); > if (reader == null) throw new IllegalArgumentException("reader must not > be null"); > {code} -- This message is automatically generated by JIRA. - You can reply to this email to add a comment to the issue online. --------------------------------------------------------------------- To unsubscribe, e-mail: java-dev-unsubscr...@lucene.apache.org For additional commands, e-mail: java-dev-h...@lucene.apache.org