opensearch-project / neural-search

Plugin that adds dense neural retrieval into the OpenSearch ecosytem
Apache License 2.0
56 stars 58 forks source link

[Part 3] Concurrent segment search bug in Sorting #808

Open vibrantvarun opened 4 days ago

vibrantvarun commented 4 days ago

Description

Cater the fix for https://github.com/opensearch-project/neural-search/issues/799 in Sorting.

This PR contains some cherry pick changes from commit https://github.com/opensearch-project/neural-search/pull/800 Therefore Classes to review are

  1. HybridCollectorManager
  2. HybridTopFieldDocComparator
  3. HybridScoreDocsMerger
  4. TopDocsMerger
  5. TopDocsMergerTests
  6. HybridScoreDocsMergerTests
  7. HybridQuerySortIT

Rest all class are cherry picked from the commit so please ignore them.

Issues Resolved

https://github.com/opensearch-project/neural-search/issues/799 https://github.com/opensearch-project/neural-search/issues/507

Check List

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.