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] Add parent join support for lucene knn #1230

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport a496926 from #1182

@heemin32
Copy link
Collaborator

Pending for opensearch-project/OpenSearch#10360

@heemin32 heemin32 force-pushed the backport/backport-1182-to-2.x branch from e366ffd to 288f120 Compare October 11, 2023 21:16
@heemin32
Copy link
Collaborator

Pending #1211

Call DiversifyingChildren[Byte|Float]KnnVectorQuery for nested field so that k number of parent document can be returned in search result

Signed-off-by: Heemin Kim <[email protected]>
(cherry picked from commit a496926)
@heemin32 heemin32 force-pushed the backport/backport-1182-to-2.x branch from 288f120 to b7539b9 Compare November 7, 2023 17:37
Copy link

codecov bot commented Nov 7, 2023

Codecov Report

Merging #1230 (b7539b9) into 2.x (785e477) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                2.x    #1230      +/-   ##
============================================
+ Coverage     85.12%   85.16%   +0.04%     
- Complexity     1187     1188       +1     
============================================
  Files           155      155              
  Lines          4881     4875       -6     
  Branches        450      451       +1     
============================================
- Hits           4155     4152       -3     
+ Misses          530      527       -3     
  Partials        196      196              
Files Coverage Δ
...n/java/org/opensearch/knn/common/KNNConstants.java 94.11% <ø> (ø)
...rg/opensearch/knn/index/query/KNNQueryFactory.java 86.27% <100.00%> (-1.45%) ⬇️

... and 1 file with indirect coverage changes

@heemin32 heemin32 merged commit f32f25a into 2.x Nov 7, 2023
71 checks passed
@github-actions github-actions bot deleted the backport/backport-1182-to-2.x branch November 7, 2023 18:15
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