Re: [PR] Remove `supportsConcurrency` flag from top-docs collector managers. [lucene]

2024-11-05 Thread via GitHub
jpountz merged PR #13977: URL: https://github.com/apache/lucene/pull/13977 -- 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.apa

[PR] Remove `supportsConcurrency` flag from top-docs collector managers. [lucene]

2024-11-04 Thread via GitHub
jpountz opened a new pull request, #13977: URL: https://github.com/apache/lucene/pull/13977 Our collector managers have a `supportsConcurrency` flag to optimize the case when they are used in a single thread. This PR proposes to remove this flag now that the optimization doesn't do much as

Re: [PR] Remove `supportsConcurrency` flag from top-docs collector managers. [lucene]

2024-11-04 Thread via GitHub
jpountz commented on PR #13977: URL: https://github.com/apache/lucene/pull/13977#issuecomment-2455325803 Here is the output of luceneutil on wikibigall with `-searchConcurrency 0`. Despite the removal of optimizations for the case when there is no search concurrency, all variations look lik