ChrisHegarty commented on issue #13818: URL: https://github.com/apache/lucene/issues/13818#issuecomment-2371868712
> It looks like the smallest two segment could have been merged - this is what the assertion complains about. > > ``` > 1> 0 info.maxDoc()=7431, info.getDelCount()=0 > 1> 1 info.maxDoc()=27811, info.getDelCount()=0 > ``` > > Merging these will be much smaller than `maxDocsPerSegment=49637`, so they are certainly a candidate for merging. But doesn't look like too much of a problem - a little less aggressive than it possibly could be. Ok, so these segments won't be merged because the size of the resulting segment would not be at least 50% larger than the biggest input segment. After looking, I don't see a product bug here. But rather a test issue that does not account for a corner case. @mikemccand @jpountz ?? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
