Skip to content

[Backport 2.x] Add parent join support for lucene knn#1230

Merged
heemin32 merged 1 commit into2.xfrom
backport/backport-1182-to-2.x
Nov 7, 2023
Merged

[Backport 2.x] Add parent join support for lucene knn#1230
heemin32 merged 1 commit into2.xfrom
backport/backport-1182-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

Backport a496926 from #1182

@heemin32
Copy link
Copy Markdown
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
Copy Markdown
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 <heemin@amazon.com>
(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
@codecov
Copy link
Copy Markdown

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
@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