Deprecate Analyzer.tokenStream
------------------------------

                 Key: LUCENE-1678
                 URL: https://issues.apache.org/jira/browse/LUCENE-1678
             Project: Lucene - Java
          Issue Type: Bug
          Components: Analysis
            Reporter: Michael McCandless
            Assignee: Michael McCandless
            Priority: Minor
             Fix For: 2.9


The addition of reusableTokenStream to the core analyzers unfortunately broke 
back compat of external subclasses:

    
http://www.nabble.com/Extending-StandardAnalyzer-considered-harmful-td23863822.html

On upgrading, such subclasses would silently not be used anymore, since 
Lucene's indexing invokes reusableTokenStream.

I think we should should at least deprecate Analyzer.tokenStream, today, so 
that users see deprecation warnings if their classes override this method.  But 
going forward when we want to change the API of core classes that are extended, 
I think we have to  introduce entirely new classes, to keep back compatibility.

-- 
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

Reply via email to