[ https://issues.apache.org/jira/browse/LUCENE-1911?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12756160#action_12756160 ]
Uwe Schindler commented on LUCENE-1911: --------------------------------------- I will post a new patch later with some added tests. I will also add SortedVIntList to the cacheable implementations. Maybe in 3.1 we could add a DocIdSet.isCacheable() method (like the proposed isRandomAccess()), that defaults to false. The 4 utils impls of DocIdSet could then return true here. This would be cleaner than the instanceof check. Or do somebody want isCacheable to DocIdSet added for 2.9, too? > When using QueryWrapperFilter with CachingWrapperFilter, QueryWrapperFilter > returns a DocIdSet that creates a Scorer, which gets cached rather than a bit > set > ------------------------------------------------------------------------------------------------------------------------------------------------------------- > > Key: LUCENE-1911 > URL: https://issues.apache.org/jira/browse/LUCENE-1911 > Project: Lucene - Java > Issue Type: Bug > Reporter: Mark Miller > Assignee: Uwe Schindler > Fix For: 2.9 > > Attachments: LUCENE-1911.patch > > > there is a large performance cost to this. > The old impl for this type of thing, QueryFilter, recommends : > @deprecated use a CachingWrapperFilter with QueryWrapperFilter > The deprecated QueryFilter itself also suffers from the problem because its > now implemented using a CachingWrapperFilter and QueryWrapperFilter. > see > http://search.lucidimagination.com/search/document/7f54715f14b8b7a/lucene_2_9_0rc4_slower_than_2_4_1 -- 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