Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Backport 2.x] Changed format for hybrid query results to a single list of scores with delimiter #266

Merged
merged 1 commit into from
Aug 29, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 75b59cd from #259

…th delimiter (#259)

* Changed approach for storing hybrid query results from compound top docs to signle list of scores with delimiter

Signed-off-by: Martin Gaievski <[email protected]>
(cherry picked from commit 75b59cd)
@codecov
Copy link

codecov bot commented Aug 29, 2023

Codecov Report

Merging #266 (e250a85) into 2.x (185050a) will decrease coverage by 0.85%.
The diff coverage is 76.55%.

@@             Coverage Diff              @@
##                2.x     #266      +/-   ##
============================================
- Coverage     86.29%   85.44%   -0.85%     
- Complexity      340      370      +30     
============================================
  Files            29       30       +1     
  Lines           992     1079      +87     
  Branches        153      165      +12     
============================================
+ Hits            856      922      +66     
- Misses           70       83      +13     
- Partials         66       74       +8     
Files Changed Coverage Δ
...neuralsearch/processor/NormalizationProcessor.java 80.00% <60.00%> (-3.88%) ⬇️
...lsearch/search/query/HybridQueryPhaseSearcher.java 71.42% <62.85%> (-2.16%) ⬇️
...arch/processor/NormalizationProcessorWorkflow.java 81.13% <72.97%> (-6.87%) ⬇️
...r/normalization/L2ScoreNormalizationTechnique.java 83.33% <75.00%> (ø)
...rmalization/MinMaxScoreNormalizationTechnique.java 84.61% <80.00%> (ø)
...ralsearch/processor/combination/ScoreCombiner.java 91.83% <85.71%> (ø)
...arch/search/util/HybridSearchResultFormatUtil.java 85.71% <85.71%> (ø)
...search/neuralsearch/processor/CompoundTopDocs.java 92.30% <92.30%> (ø)
...earch/processor/normalization/ScoreNormalizer.java 100.00% <100.00%> (+20.00%) ⬆️

@martin-gaievski martin-gaievski merged commit e564d4d into 2.x Aug 29, 2023
@github-actions github-actions bot deleted the backport/backport-259-to-2.x branch August 29, 2023 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants