lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Erik Hatcher <>
Subject Re: Caching filter wrapper
Date Tue, 16 Sep 2003 21:05:03 GMT
I've added the proposed CachingWrapperFilter using QueryFilter's method 
of caching along with the fix Otis put in this morning to avoid an NPE 
when using remote searching.

See the @todo's there to see if there is more work to be done.  Using 
QueryFilter or this caching one through remote searcher will void any 
caching - is this of concern here?  Do we need to provide a 
user-definable Map implementation for the cache store as an option?

I'll write up some javadocs for this once we've ironed out the 


On Tuesday, September 16, 2003, at 03:28  PM, Bruce Ritchie wrote:

> Erik Hatcher wrote:
>> Cool.... I'll work on adding an implementation then.  But what would 
>> be the key to the map if not the IndexReader instance?  It ought to 
>> be something related to that at least for the scenario's where a 
>> single filter instance is being used over multiple indices.  Or would 
>> simply two different constructors be enough (one taking a Filter and 
>> defaulting to a WeakHashMap, and the other taking a Filter and a Map 
>> to use), and still use IndexReader as the key?
> Well, seeing as how things are resolving on the other half of this 
> thread, I'll take back my concern about tying the caching to a reader 
> instance (since I've come up with a decent solution which will allow 
> me to use a single reader).
> Regards,
> Bruce Ritchie
> <smime.p7s>

View raw message