javanna commented on PR #16433: URL: https://github.com/apache/lucene/pull/16433#issuecomment-5269495849
Hi, thanks for posting this. I had tried the same approach prior to opening the related issue. But like I wrote in the issue: > There may be value in parallelizing execution, but besides exposing the ability for users to provide an executor like we do with ordinary search, a refactor is needed to properly support inter-segment concurrency in monitor. Actually, your current changes are going to trigger assertion errors because the monitor collector managers expect a single slice and don't support concurrent execution. There is more work needed here to properly support concurrent execution within monitor. -- 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]
