[Backport 3.2] [GRPC] Extend transport-grpc module to support GRPC KNN queries#2841
Merged
[Backport 3.2] [GRPC] Extend transport-grpc module to support GRPC KNN queries#2841
Conversation
* [GRPC] Extend transport-grpc plugin to support GRPC KNN queries and upgrade guava Signed-off-by: Karen Xu <karenxyr@gmail.com> * use @Utility and refactor into smaller methods Signed-off-by: Karen Xu <karenxyr@gmail.com> * Force bundle guava dependencies Signed-off-by: Karen Xu <karenxyr@gmail.com> * Force bundle failure access only not guava, and upgrade guava to match security plugin version Signed-off-by: Karen Xu <karenxyr@gmail.com> * Downgrade failure access back and dont force Signed-off-by: Karen Xu <karenxyr@gmail.com> * force deps for tests Signed-off-by: Karen Xu <karenxyr@gmail.com> * use 32.1.3 original Signed-off-by: Karen Xu <karenxyr@gmail.com> * move from changlog to 3.2. release notes Signed-off-by: Karen Xu <karenxyr@gmail.com> * Revert "move from changlog to 3.2. release notes" This reverts commit e996916. Signed-off-by: Karen Xu <karenxyr@gmail.com> * Revert "Revert "move from changlog to 3.2. release notes"" This reverts commit 590361c. Signed-off-by: Karen Xu <karenxyr@gmail.com> * force resolution for all, not just tests, and resolve changelog conflicts Signed-off-by: Karen Xu <karenxyr@gmail.com> * filter out jacoco Signed-off-by: Karen Xu <karenxyr@gmail.com> * exclude guava and failure access from transport-grpc Signed-off-by: Karen Xu <karenxyr@gmail.com> * upgrade failureaccess and guava to match core, and make guava a test only dep Signed-off-by: Karen Xu <karenxyr@gmail.com> * fix unit test Signed-off-by: Karen Xu <karenxyr@gmail.com> --------- Signed-off-by: Karen Xu <karenxyr@gmail.com> Signed-off-by: Doo Yong Kim <0ctopus13prime@gmail.com> Co-authored-by: Navneet Verma <navneev@amazon.com> Co-authored-by: Doo Yong Kim <0ctopus13prime@gmail.com> (cherry picked from commit 49ff9a8)
opensearch-trigger-bot
bot
requested review from
0ctopus13prime,
VijayanB,
Vikasht34,
heemin32,
jmazanec15,
junqiu-lei,
luyuncheng,
martin-gaievski,
naveentatikonda,
navneet1v,
ryanbogan,
shatejas and
vamshin
as code owners
vamshin
approved these changes
Aug 8, 2025
navneet1v
approved these changes
Aug 8, 2025
navneet1v
approved these changes
Aug 8, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Backport 49ff9a8 from #2817