[ https://issues.apache.org/jira/browse/LUCENE-781?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12515384 ]
Michael Busch commented on LUCENE-781: -------------------------------------- If there are no objections against separating MultiReader into two classes I would like to commit my patch in a day or two. I think the javadocs look fine, the class MultiSegmentReader does not appear there. One thing I'm not sure about: Should MultiReader throw an UnsupportedOperationException or simply return false when isOptimized() is called? > NPE in MultiReader.isCurrent() and getVersion() > ----------------------------------------------- > > Key: LUCENE-781 > URL: https://issues.apache.org/jira/browse/LUCENE-781 > Project: Lucene - Java > Issue Type: Bug > Components: Index > Reporter: Daniel Naber > Assignee: Michael Busch > Attachments: lucene-781.patch, multireader.diff, multireader.diff, > multireader_test.diff, multireader_test.diff > > > I'm attaching a fix for the NPE in MultiReader.isCurrent() plus a testcase. > For getVersion(), we should throw a better exception that NPE. I will commit > unless someone objects or has a better idea. -- 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]