lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michael McCandless <>
Subject Re: Allow IndexReader to take ownership of Directory
Date Tue, 18 Nov 2008 20:37:18 GMT

I think this makes sense.

But: I think we'd need to add incRef/decRef to Directory?  And fix the  
newly added logic in DirectoryIndexReader that now clones the dir  
during reopen (because it's hardwired to only work with FSDir).


Mark Miller wrote:

> Does anyone object to making IndexReader
> private static IndexReader open(final Directory directory, final  
> boolean closeDirectory, final IndexDeletionPolicy deletionPolicy,  
> final IndexCommit commit, final boolean readOnly) throws        
> CorruptIndexException, IOException {
>   return, closeDirectory,  
> deletionPolicy, commit, readOnly);
> }
> public? I think it can be convenient to be able to both control the  
> closing of the Directory yourself, as well as give the Directory to  
> an IndexReader to manage.
> If you have created a DirectoryFactory to supply your IndexReaders,  
> you will want to supply external Directory implementations to the  
> IndexReaders, and its quite possible that you then want the  
> IndexReader to take over the lifecycle of the Directory for you  
> rather than having to juggle the two separately
> - Mark
> ---------------------------------------------------------------------
> To unsubscribe, e-mail:
> For additional commands, e-mail:

To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message