-
Notifications
You must be signed in to change notification settings - Fork 72
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] Backport 515 to 2.x #528
[Backport 2.x] Backport 515 to 2.x #528
Conversation
…ch-project#515) * Reformatting test package Signed-off-by: Varun Jain <[email protected]> * Initial commit of BWC Test Signed-off-by: Varun Jain <[email protected]> * Text Search Signed-off-by: Varun Jain <[email protected]> * Fixing bug Signed-off-by: Varun Jain <[email protected]> * Text Search bug fix Signed-off-by: Varun Jain <[email protected]> * Adding windows platform in bwc Signed-off-by: Varun Jain <[email protected]> * Adding windows platform in bwc Signed-off-by: Varun Jain <[email protected]> * Rolling Upgrade tests Signed-off-by: Varun Jain <[email protected]> * Bux Fix in rolling upgrade Signed-off-by: Varun Jain <[email protected]> * Bug Fix Rolling Upgrade Signed-off-by: Varun Jain <[email protected]> * Fixing Flaky tests Signed-off-by: Varun Jain <[email protected]> * Updating BWC version to latest Signed-off-by: Varun Jain <[email protected]> * Fixing bwc test Signed-off-by: Varun Jain <[email protected]> * Semantic Search Signed-off-by: Varun Jain <[email protected]> * Bug Fix Signed-off-by: Varun Jain <[email protected]> * Debugging Signed-off-by: Varun Jain <[email protected]> * Bug Fix Signed-off-by: Varun Jain <[email protected]> * Increase memory in nodes Signed-off-by: Varun Jain <[email protected]> * Removing extra logging Signed-off-by: Varun Jain <[email protected]> * Cleaning up Signed-off-by: Varun Jain <[email protected]> * Updating Pipeline Configuration Signed-off-by: Varun Jain <[email protected]> * Remove KNN delete models Signed-off-by: Varun Jain <[email protected]> * Remove unecessary KNN code Signed-off-by: Varun Jain <[email protected]> * Addressing comments of naveen Signed-off-by: Varun Jain <[email protected]> * Addressing comments of naveen Signed-off-by: Varun Jain <[email protected]> * Addressing comments of Naveen Signed-off-by: Varun Jain <[email protected]> * Addressing martin comments Signed-off-by: Varun Jain <[email protected]> * Addressing comments of martin Signed-off-by: Varun Jain <[email protected]> * Apply formatting.xml in all lines Signed-off-by: Varun Jain <[email protected]> * Removing extra spacess from formatting.gradle Signed-off-by: Varun Jain <[email protected]> * Addressing martin comment Signed-off-by: Varun Jain <[email protected]> * Addressing Jack comments Signed-off-by: Varun Jain <[email protected]> * Addressing Jack comments Signed-off-by: Varun Jain <[email protected]> * Addressing Jack's comments Signed-off-by: Varun Jain <[email protected]> * Fixing Test cases Signed-off-by: Varun Jain <[email protected]> * Addressing comments of Navneet Signed-off-by: Varun Jain <[email protected]> * Addressing comments of Navneet Signed-off-by: Varun Jain <[email protected]> * Addressing comments of Navneet Signed-off-by: Varun Jain <[email protected]> * Addressing comments of Navneet Signed-off-by: Varun Jain <[email protected]> * Removing extra parameter from createPipelineProcessor Signed-off-by: Varun Jain <[email protected]> * Fixing bug Signed-off-by: Varun Jain <[email protected]> * Increasing number of shards Signed-off-by: Varun Jain <[email protected]> * Bug fix of load model id Signed-off-by: Varun Jain <[email protected]> * Changing names of tests to TextEmbeddingPRocessor Signed-off-by: Varun Jain <[email protected]> * Updating indexes and replicas Signed-off-by: Varun Jain <[email protected]> --------- Signed-off-by: Varun Jain <[email protected]>
Signed-off-by: Varun Jain <[email protected]>
Signed-off-by: Varun Jain <[email protected]>
Signed-off-by: Varun Jain <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 2.x #528 +/- ##
=========================================
Coverage 85.18% 85.18%
Complexity 541 541
=========================================
Files 40 40
Lines 1580 1580
Branches 248 248
=========================================
Hits 1346 1346
Misses 117 117
Partials 117 117 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Varun Jain <[email protected]>
Signed-off-by: Varun Jain <[email protected]>
Signed-off-by: Varun Jain <[email protected]>
We are keeping a track on why BWC rolling upgrade tests are failing for BWC version <=2.10. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
add this in the description |
Signed-off-by: Varun Jain <[email protected]>
Description
Backport 515 manually to 2.x
We are keeping a track on why BWC rolling upgrade tests are failing for BWC version <=2.10.
#529
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.