javanna commented on PR #16091: URL: https://github.com/apache/lucene/pull/16091#issuecomment-5282165592
Heads up: I reworked the approach to be less memory costly, at the cost of using atomic data structures. That adds some overhead for the sequential case where an executor is not provided to the index searcher. I find that that is an ok compromise though, because we want search concurrency to be a first class citizen and that's kind of the whole purpose of removing the deprecated `search(Query, Collector)` method. I am looking for reviews and plan on merging this next week if there are no objections. -- 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]
