Migrate KNN serialization interfaces #20513
Migrate KNN serialization interfaces #20513ajleong623 wants to merge 2 commits intoopensearch-project:mainfrom
Conversation
Signed-off-by: Anthony Leong <anthonyleong@Anthonys-MacBook-Air.local>
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the
✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #20513 +/- ##
============================================
+ Coverage 73.19% 73.36% +0.16%
- Complexity 71975 72145 +170
============================================
Files 5796 5798 +2
Lines 329539 329654 +115
Branches 47465 47491 +26
============================================
+ Hits 241220 241842 +622
+ Misses 69005 68434 -571
- Partials 19314 19378 +64 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Signed-off-by: Anthony Leong <anthonyleong@Anthonys-MacBook-Air.local>
|
❌ Gradle check result for db4eac6: FAILURE Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change? |
Description
Adds knn interfaces that are shared between knn plugins https://github.com/opensearch-project/k-NN/tree/main and https://github.com/opensearch-project/opensearch-jvector/tree/main to core. This PR focuses on the interfaces relating to serialization logic.
Related Issues
#20050
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.