[ https://issues.apache.org/jira/browse/LUCENE-6165?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14269330#comment-14269330 ]
Robert Muir commented on LUCENE-6165: ------------------------------------- It also has no state (unlike slowwrapper). No need for overengineering here. > Change merging APIs to work on CodecReader instead of LeafReader > ---------------------------------------------------------------- > > Key: LUCENE-6165 > URL: https://issues.apache.org/jira/browse/LUCENE-6165 > Project: Lucene - Core > Issue Type: Bug > Reporter: Robert Muir > Attachments: LUCENE-6165.patch > > > Patch factors out "reader based on codec apis" and changes all merge > policy/addIndexes apis to use this. > If you want to do slow wrapping, you can still do it, just use > SlowCodecReaderWrapper.wrap(LeafReader) yourself (versus SegmentMerger doing > it always if its not a SegmentReader). > Also adds FilterCodecReader, to make it easier to start efficiently filtering > on merge. I cutover all the index splitters to this. This means they should > be much much faster with this patch, they just change the deletes as you > expect, and the merge is as optimal as a normal one. > In other places, for now I think we should just do a rote conversion with > SlowCodecReaderWrapper.wrap. Its no slower than today, just explicit, and we > can incrementally fix them to do the right thing in the future rather than > all at once. -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@lucene.apache.org For additional commands, e-mail: dev-h...@lucene.apache.org