rmuir commented on pull request #728: URL: https://github.com/apache/lucene/pull/728#issuecomment-1058821263
I'm suspicious of the reported performance improvement based on looking at your benchmark output, I don't think its realistic. Looks like nothing else was indexed in any other way (docvalues/postings/etc), nobody ever called reopen() to force any flushes, so with the benchmark you ran, IW just wrote one big segment, avoiding all merging. So everything looks fantastic on paper, but this isn't realistic. It is easy to run into the same trap when benchmarking e.g. stored fields and other things. But it isn't really a performance improvement. -- 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
