[ https://issues.apache.org/jira/browse/LUCENE-986?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12528225 ]
Michael Busch commented on LUCENE-986: -------------------------------------- > one hitch might be that SegmentReader.get is public, and in order to keep > supporting it, SegmentReader still needs to have/inherit the same segment > info > and directory owning/locking code that we want to move out of IndexReader (so > just putting it MultiSegmentReader won't fly unless we kill that public > method). OK, I implemented the DirectoryIndexReader approach. Also because I'm not sure about the performance characteristics of a MultiSegmentReader acting as a SegmentReader. I'd like to commit this rather soon. A review of the patch would be highly appreciated. > Refactor segmentInfos from IndexReader into its subclasses > ---------------------------------------------------------- > > Key: LUCENE-986 > URL: https://issues.apache.org/jira/browse/LUCENE-986 > Project: Lucene - Java > Issue Type: Improvement > Components: Index > Reporter: Michael Busch > Assignee: Michael Busch > Priority: Minor > Fix For: 2.3 > > Attachments: lucene-986.patch > > > References to segmentInfos in IndexReader cause different kinds of problems > for subclasses of IndexReader, like e. g. MultiReader. > Only subclasses of IndexReader that own the index directory, namely > SegmentReader and MultiSegmentReader, should have a SegmentInfos object > and be able to access it. > Further information: > http://www.gossamer-threads.com/lists/lucene/java-dev/51808 > http://www.gossamer-threads.com/lists/lucene/java-user/52460 > A part of the refactoring work was already done in LUCENE-781 -- 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: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]