-
Notifications
You must be signed in to change notification settings - Fork 38
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
Allow to replicate object to other nodes in control.EvacuateShard
#1800
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1800 +/- ##
==========================================
+ Coverage 33.41% 33.45% +0.04%
==========================================
Files 352 352
Lines 23962 23969 +7
==========================================
+ Hits 8008 8020 +12
+ Misses 15287 15283 -4
+ Partials 667 666 -1
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
} | ||
} | ||
|
||
var res replicatorResult |
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.
what if nodes
is empty after the current node exclusion?
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.
The res.count == 0
after HandleTask
and we will return an error.
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.
HandleTask
has some debug logging even with an empty task so i meant it could be cached earlier, but ok
Signed-off-by: Evgenii Stratonikov <evgeniy@morphbits.ru>
…vacuateShard RPC Signed-off-by: Evgenii Stratonikov <evgeniy@morphbits.ru>
… parameters Signed-off-by: Evgenii Stratonikov <evgeniy@morphbits.ru>
Related #1105