mdmarshmallow commented on PR #13951: URL: https://github.com/apache/lucene/pull/13951#issuecomment-2521888233
Sure @mikemccand , added an issue [here](https://github.com/mikemccand/luceneutil/issues/320). I also was able to test my change by just doing two separate runs and manually changing the test to point at whichever version I needed. My test setup involved hacking `Indexer#java` in `luceneutil` a bit to run 2 index writers instead of one. For mainline, I gave both index writers a 100 MB buffer, and for the candidate, I gave the ram manager a 200 MB buffer and passed that into the index writers instead. Both index writers were indexing the exact same dataset (`wikimedium1m`). I found that there was pretty much no change in indexing throughput (20.5 GB/hour for the candidate and 20.1 GB/hour for baseline). I can post more detailed results if anyone is interested. I realize that this is not a very realistic scenario, so I will try benchmarking with more writers and also skew the indexing rates of each writer to see how the change performs. -- 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
