-
Notifications
You must be signed in to change notification settings - Fork 113
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
Adding Backward Compatibility Tests for k-NN #226
Adding Backward Compatibility Tests for k-NN #226
Conversation
@naveentatikonda do we need those binary files to be checked in? if not, can you add it to .gitignore files? |
Yes Vijay, we need to check-in the binary files. We need them to run the BWCTests. Those are the gradle assemble zips of previous k-NN plugin versions and libKNNIndexV2_0_11.so |
@naveentatikonda could you add in the description whats actions the tests are performing to confirm a change is backwards compatible? |
3c0e4b9
to
24add37
Compare
src/test/java/org/opensearch/knn/bwc/KNNBackwardsCompatibilityIT.java
Outdated
Show resolved
Hide resolved
src/test/java/org/opensearch/knn/bwc/KNNBackwardsCompatibilityIT.java
Outdated
Show resolved
Hide resolved
src/test/java/org/opensearch/knn/bwc/KNNBackwardsCompatibilityIT.java
Outdated
Show resolved
Hide resolved
src/test/java/org/opensearch/knn/bwc/KNNBackwardsCompatibilityIT.java
Outdated
Show resolved
Hide resolved
src/test/java/org/opensearch/knn/bwc/KNNBackwardsCompatibilityIT.java
Outdated
Show resolved
Hide resolved
src/test/java/org/opensearch/knn/bwc/KNNBackwardsCompatibilityIT.java
Outdated
Show resolved
Hide resolved
src/test/java/org/opensearch/knn/bwc/KNNBackwardsCompatibilityIT.java
Outdated
Show resolved
Hide resolved
src/test/java/org/opensearch/knn/bwc/KNNBackwardsCompatibilityIT.java
Outdated
Show resolved
Hide resolved
src/test/java/org/opensearch/knn/bwc/KNNBackwardsCompatibilityIT.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Naveen Tatikonda <navtat@amazon.com>
f98e570
to
201f105
Compare
@jmazanec15 @VijayanB updated the code as per our discussion offline |
Codecov Report
@@ Coverage Diff @@
## main #226 +/- ##
=========================================
Coverage 83.21% 83.21%
Complexity 864 864
=========================================
Files 123 123
Lines 3777 3777
Branches 358 358
=========================================
Hits 3143 3143
Misses 474 474
Partials 160 160 Continue to review full report at Codecov.
|
Signed-off-by: Naveen Tatikonda <navtat@amazon.com>
src/test/java/org/opensearch/knn/bwc/KNNBackwardsCompatibilityIT.java
Outdated
Show resolved
Hide resolved
src/test/java/org/opensearch/knn/bwc/KNNBackwardsCompatibilityIT.java
Outdated
Show resolved
Hide resolved
Signed-off-by: Naveen Tatikonda <navtat@amazon.com>
9a8ff20
to
1c11d8a
Compare
* Adding Backward Compatibility Tests for k-NN Signed-off-by: Naveen Tatikonda <navtat@amazon.com> * Delete unnecessary Binary files Signed-off-by: Naveen Tatikonda <navtat@amazon.com> * Updated Developer Guide and refactored code Signed-off-by: Naveen Tatikonda <navtat@amazon.com>
* Adding Backward Compatibility Tests for k-NN Signed-off-by: Naveen Tatikonda <navtat@amazon.com> * Delete unnecessary Binary files Signed-off-by: Naveen Tatikonda <navtat@amazon.com> * Updated Developer Guide and refactored code Signed-off-by: Naveen Tatikonda <navtat@amazon.com> Signed-off-by: Martin Gaievski <gaievski@amazon.com>
* Adding Backward Compatibility Tests for k-NN Signed-off-by: Naveen Tatikonda <navtat@amazon.com> * Delete unnecessary Binary files Signed-off-by: Naveen Tatikonda <navtat@amazon.com> * Updated Developer Guide and refactored code Signed-off-by: Naveen Tatikonda <navtat@amazon.com>
Signed-off-by: Naveen Tatikonda navtat@amazon.com
Description
Adding Backward Compatibility Tests(Mixed Cluster test, Rolling upgrade test, Full restart upgrade test) to k-NN
Issues Resolved
#84
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.