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

Uwe Schindler commented on LUCENE-1796:
---------------------------------------

bq. I don't know if all of the Tokenizers in lucene were ever changed, but in 
any case it looks like at least some of these bugs were introduced with the 
switch to the attribute API - for example StandardTokenizer did clear it's 
reusableToken... and now it doesn't.

No one is calling clearAttributes() in trunk code, only some of them clear 
attributes before filling data in.

OK, I open another issue later and change all Tokenizers in core and contrib to 
call clearAttributes() as first call inside incrementToken()?

But in principle we could also change the indexer to call clear before each 
incrementToken() removing the need to do it in every Tokenizer.

> Speed up repeated TokenStream init
> ----------------------------------
>
>                 Key: LUCENE-1796
>                 URL: https://issues.apache.org/jira/browse/LUCENE-1796
>             Project: Lucene - Java
>          Issue Type: Improvement
>            Reporter: Mark Miller
>            Assignee: Uwe Schindler
>             Fix For: 2.9
>
>         Attachments: after.png, afterAndLucene1796.png, before.png, 
> LUCENE-1796.patch, LUCENE-1796.patch, LUCENE-1796.patch, LUCENE-1796.patch, 
> LUCENE-1796.patch
>
>
>  by caching isMethodOverridden results

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