[LUCENE-3606] Make IndexReader really read-only in Lucene 4.0

• Type: Task
• Status: Closed
• Priority: Major
• Resolution: Fixed
• Affects Version/s: 4.0-ALPHA
• Fix Version/s: 3.6, 4.0-ALPHA
• Component/s: core/index
• Lucene Fields: New
• Assignee: Uwe Schindler
• Reporter: Uwe Schindler
• Created: 28/Nov/11 17:09
• Updated: 28/Aug/22 13:02
• Resolved: 09/Dec/11 17:55

Description

As we change API completely in Lucene 4.0 we are also free to remove read-write access and commits from IndexReader. This code is so hairy and buggy (as investigated by Robert and Mike today) when you work on SegmentReader level but forget to flush in the DirectoryReader, so its better to really make IndexReaders readonly.

Currently with IndexReader you can do things like:

• delete/undelete Documents -> Can be done by with IndexWriter, too (using deleteByQuery)
• change norms -> this is a bad idea in general, but when we remove norms at all and replace by DocValues this is obsolete already. Changing DocValues should also be done using IndexWriter in trunk (once it is ready)

Issue Links

• incorporates: LUCENE-3620 FilterIndexReader does not override all of IndexReader methods