-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Fix weighted routing metadata deserialization error during node restart #4691
Fix weighted routing metadata deserialization error during node restart #4691
Conversation
Signed-off-by: Anshu Agarwal <[email protected]>
Gradle Check (Jenkins) Run Completed with:
|
Signed-off-by: Anshu Agarwal <[email protected]>
Gradle Check (Jenkins) Run Completed with:
|
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.
Please lets add tests here
Gradle Check (Jenkins) Run Completed with:
|
Gradle Check (Jenkins) Run Completed with:
|
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, thanks @anshu1106
Gradle Check (Jenkins) Run Completed with:
|
Build is unstable due to unrelated test failures. Building again
|
Signed-off-by: Anshu Agarwal <[email protected]>
636afab
to
cc83386
Compare
Gradle Check (Jenkins) Run Completed with:
|
Codecov Report
@@ Coverage Diff @@
## main #4691 +/- ##
============================================
+ Coverage 70.71% 70.73% +0.02%
- Complexity 57569 57599 +30
============================================
Files 4660 4660
Lines 276601 276600 -1
Branches 40307 40307
============================================
+ Hits 195596 195660 +64
+ Misses 64771 64696 -75
- Partials 16234 16244 +10
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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.
thx!
Gradle Check (Jenkins) Run Completed with:
|
…rt (opensearch-project#4691) * Fix weighted routing metadata deserialization error during node restart Signed-off-by: Anshu Agarwal <[email protected]>
…rt (opensearch-project#4691) * Fix weighted routing metadata deserialization error during node restart Signed-off-by: Anshu Agarwal <[email protected]>
* Weighted round-robin scheduling policy for shard coordination traffic… (#4241) * Add PUT api to update shard routing weights (#4272) * Add GET api to get shard routing weights (#4275) * Fix weighted routing metadata deserialization error during node restart (#4691) * Delete API for weighted round robin search routing (#4400) * Mark apis experimental Signed-off-by: Anshu Agarwal <[email protected]>
* Weighted round-robin scheduling policy for shard coordination traffic… (#4241) * Add PUT api to update shard routing weights (#4272) * Add GET api to get shard routing weights (#4275) * Fix weighted routing metadata deserialization error during node restart (#4691) * Delete API for weighted round robin search routing (#4917 * Mark apis experimental Signed-off-by: Anshu Agarwal <[email protected]>
…rt (opensearch-project#4691) * Fix weighted routing metadata deserialization error during node restart Signed-off-by: Anshu Agarwal <[email protected]>
Signed-off-by: Anshu Agarwal [email protected]
Description
Deserialization of weighted routing metadata is failing during node restart. This PR fixes the serialization error
Issues Resolved
#4690
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.