neural-search
neural-search copied to clipboard
Implement parallel execution of sub-queries for hybrid search
Description
- Add new thread pool to schedule tasks that are related to hybrid query execution
- Register executor builders with Plugin
- Use Lucene's Task Executor to execute and collect results
- Parallelize Query re-write
- Parallelize score supplier creation
- Parallelize build hybrid scores
Issues Resolved
#279
Check List
- [x] New functionality includes testing.
- [x] All tests pass
- [ ] New functionality has been documented.
- [ ] New functionality has javadoc added
- [x] Commits are signed as per the DCO using --signoff
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. For more information on following Developer Certificate of Origin and signing off your commits, please check here.
Codecov Report
Attention: Patch coverage is 83.78378% with 12 lines in your changes are missing coverage. Please review.
Please upload report for BASE (
feature/parallelize-hybrid-search@806042c). Learn more about missing BASE report.
:exclamation: Current head 2750ad4 differs from pull request most recent head bc6b885
Please upload reports for the commit bc6b885 to get more accurate results.
Additional details and impacted files
@@ Coverage Diff @@
## feature/parallelize-hybrid-search #749 +/- ##
====================================================================
Coverage ? 84.89%
Complexity ? 812
====================================================================
Files ? 65
Lines ? 2490
Branches ? 410
====================================================================
Hits ? 2114
Misses ? 213
Partials ? 163
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@navneet1v Can you take a look at one more time? Thanks
@VijayanB I will reviewing the PR today in sometime.