[ https://issues.apache.org/jira/browse/LUCENE-3736?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13202497#comment-13202497 ]
Uwe Schindler commented on LUCENE-3736: --------------------------------------- If somebody wants to commit the changes, do it. I want to be out of that, as I don't want to be responsible when somebody opens an issue that says "Builder pattern without chaining is broken" (which it is). Do what you want but keep me away from it! Just a suggestion: - As noted before, a Builder class with a build() method without chaining violates the pattern, so please rename it! I unassigned from this issue and will take care of some minor AtomicReaderContext changes (to remove one more utility method from ReaderUtil thats useless, if the API would implement what Javadocs say - I am talking about leaves()). > ParallelReader is now atomic, rename to ParallelAtomicReader and also add a > ParallelCompositeReader (that requires LogDocMergePolicy to have identical > subreader structure) > --------------------------------------------------------------------------------------------------------------------------------------------------------------------------- > > Key: LUCENE-3736 > URL: https://issues.apache.org/jira/browse/LUCENE-3736 > Project: Lucene - Java > Issue Type: Sub-task > Components: core/index > Reporter: Uwe Schindler > Fix For: 4.0 > > Attachments: LUCENE-3733.patch, LUCENE-3736.patch, LUCENE-3736.patch, > LUCENE-3736.patch > > > The plan is: > - Move all subreaders to ctor (builder-like API. First build reader-set, then > call build) > - Rename ParallelReader to ParallelAtomicReader > - Add a ParallelCompositeReader with same builder API, but taking any > CompositeReader-set and checks them that they are aligned (docStarts > identical). The subreaders are ParallelAtomicReaders. -- This message is automatically generated by JIRA. If you think it was sent incorrectly, please contact your JIRA administrators: https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org