[
https://issues.apache.org/jira/browse/LUCENE-937?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12507334
]
Mark Miller commented on LUCENE-937:
------------------------------------
Well this is embarrassing. I messed up the implementation of the iterator
approach. Egg on my face moment. At least now things make more sense.
LinkedList using an iterator is as fast as ArrayList(16). It does not appear to
be any faster, but it is more memory efficient. Michael was absolutely right
and the mistake is the direct access over the iterator.
Sorry about the major misdirection on this...youthful exuberance always gets
the best of me...<g>
I suggest the change is just to change from get() to Iterator usage.
> Make CachingTokenFilter faster
> ------------------------------
>
> Key: LUCENE-937
> URL: https://issues.apache.org/jira/browse/LUCENE-937
> Project: Lucene - Java
> Issue Type: Improvement
> Reporter: Mark Miller
> Priority: Minor
> Attachments: CachingTokenFilter.patch
>
>
> The wrong data structure was used for the CachingTokenFilter. It should be an
> ArrayList rather than a LinkedList. There is a noticeable difference in speed.
--
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
