-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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] Fix remote shards balance #16972
[Backport 2.x] Fix remote shards balance #16972
Conversation
* Fix remote shards balance Signed-off-by: panguixin <panguixin@bytedance.com> * add changelog Signed-off-by: panguixin <panguixin@bytedance.com> --------- Signed-off-by: panguixin <panguixin@bytedance.com> Signed-off-by: Andrew Ross <andrross@amazon.com> Co-authored-by: Andrew Ross <andrross@amazon.com> (cherry picked from commit b359dd8)
❌ Gradle check result for fc51b60: 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? |
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.
Thanks for the fix @bugmakerrrrrr!
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 2.x #16972 +/- ##
============================================
- Coverage 71.85% 71.81% -0.05%
- Complexity 65507 65512 +5
============================================
Files 5316 5316
Lines 305588 305595 +7
Branches 44516 44518 +2
============================================
- Hits 219572 219450 -122
- Misses 67674 67857 +183
+ Partials 18342 18288 -54 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Andrew Ross <andrross@amazon.com>
❌ Gradle check result for db04416: 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
backport #15335
Related Issues
Resolves #[Issue number to be closed when this PR is merged]
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.