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

Uwe Schindler commented on LUCENE-5097:
---------------------------------------

Another suggestion here:
Currently we have a crazy reuseable reader in Field.java. This one could go 
away, instead the Analyzer would store a resuseable reader in 
TokenStreamComponents/the TS cache. Field.java would be simplier as it would 
just call this method to get the TS from a String field.
                
> Add utility method to Analyzer: public final TokenStream tokenStream(String 
> fieldName,String text)
> --------------------------------------------------------------------------------------------------
>
>                 Key: LUCENE-5097
>                 URL: https://issues.apache.org/jira/browse/LUCENE-5097
>             Project: Lucene - Core
>          Issue Type: Bug
>          Components: modules/analysis
>    Affects Versions: 4.3.1
>            Reporter: Uwe Schindler
>
> It might be a good idea to remove tons of useless code from tests:
> Most people use TokenStreams and Analyzers by only passing a String, wrapped 
> by a StringReader. It would make life easier, if Analyzer would have an 
> additional public (and final!!!) method that simply does the wrapping with 
> StringReader by itsself. It might maybe not even needed to throw IOException 
> (not sure)

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

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

Reply via email to