You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@lucene.apache.org by "cyforkk (via GitHub)" <gi...@apache.org> on 2026/07/29 14:17:12 UTC
[PR] feat: Add Executor support to Monitor for concurrent search (issue #16198) [lucene]
cyforkk opened a new pull request, #16433:
URL: https://github.com/apache/lucene/pull/16433
## Summary
This PR adds support for passing an Executor to the Monitor for inter-segment concurrency in search operations.
## Changes
- Added `Executor` field to `MonitorConfiguration` with getter/setter methods
- Modified `Monitor` constructor to store the Executor from configuration
- Updated `match()` and `debug()` methods to use the Executor when creating `IndexSearcher` instances
## Usage
Users can now configure concurrent search in Monitor:
```java
MonitorConfiguration config = new MonitorConfiguration();
config.setExecutor(executor);
Monitor monitor = new Monitor(analyzer, config);
```
## Related
Fixes #16198
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
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-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
Re: [PR] feat: Add Executor support to Monitor for concurrent search (issue #16198) [lucene]
Posted by "javanna (via GitHub)" <gi...@apache.org>.
javanna closed pull request #16433: feat: Add Executor support to Monitor for concurrent search (issue #16198)
URL: https://github.com/apache/lucene/pull/16433
--
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-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
Re: [PR] feat: Add Executor support to Monitor for concurrent search (issue #16198) [lucene]
Posted by "javanna (via GitHub)" <gi...@apache.org>.
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: issues-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org
Re: [PR] feat: Add Executor support to Monitor for concurrent search (issue #16198) [lucene]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #16433:
URL: https://github.com/apache/lucene/pull/16433#issuecomment-5447885151
This PR has not had activity in the past 2 weeks, labeling it as stale. If the PR is waiting for review, notify the dev@lucene.apache.org list. Thank you for your contribution!
--
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-unsubscribe@lucene.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@lucene.apache.org
For additional commands, e-mail: issues-help@lucene.apache.org