lucene-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Michael McCandless (JIRA)" <>
Subject [jira] Commented: (LUCENE-1551) Add reopen(IndexCommit) methods to IndexReader
Date Thu, 05 Mar 2009 11:37:56 GMT


Michael McCandless commented on LUCENE-1551:

Since we moved delete to IndexWriter, I strongly feel that IndexReader SHOULD be always read-only,
so your patch makes perfect sense to me.

I agree...

In 3.0, we are switching to return a readOnly reader
by default (it's now read/write).

I know that there are still some operations that IndexWriter can't do, and real-time search
needs them,
so that's why LUCENE-1516 was created, right? But those guys don't need to open previous commit
they need latest & greatest, so the patch should work for them too.

Right, once realtime search is released, proves stable, etc.,
IndexReader should no longer need any write operations (I think?).

But in the end (lucene4, or maybe sooner?), could it be possible to move all modifying methods
to IndexWriter and let IndexReader to be what it says it is: a READER?

I think LUCENE-1516 won't land until after 3.0 (though, it's making
fast progress now, so it could be we get initial version into 2.9).

Then sometime in 3.x, we deprecate all write methods in IndexReader
and add anything missing (setNorm, undeleteAll) into IndexWriter.

Then in 4.x we would remove IndexReader's deprecated methods.

bq. Beside this totally random talk, the patch looks good, I'll try to test it in a day or
two and let you know if there are any problems.

Excellent, thanks -- I'll hold off on committing until we hear back
how your tests go!

> Add reopen(IndexCommit) methods to IndexReader
> ----------------------------------------------
>                 Key: LUCENE-1551
>                 URL:
>             Project: Lucene - Java
>          Issue Type: Improvement
>          Components: Index
>    Affects Versions: 2.9
>         Environment: Any
>            Reporter: Torin Danil
>            Assignee: Michael McCandless
>            Priority: Minor
>             Fix For: 2.9
>         Attachments: LUCENE-1551.patch
> Add reopen(IndexCommit) methods to IndexReader to be able to reopen an index on any previously
saved commit points with all advantages of LUCENE-1483.
> Similar to open(IndexCommit) & company available in 2.4.0.

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:
For additional commands, e-mail:

View raw message