antonha commented on PR #13149: URL: https://github.com/apache/lucene/pull/13149#issuecomment-1986780683
Thanks for the approval! I do however think that it would be good to prove this performance improvement in luceneutil before merging, to make the benchmark more easily reproducible than the benchmarking I have done. I have started looking into it, but will have more time for it next week. Does that sound reasonable? The only downside I see is a later merge of this PR. > @antonha could you add a `CHANGES.txt` entry explaining the optimization? Thanks! Yes, will do once I have run some luceneutil benchmarks (if we want to wait for that) -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
