Binglin Chang has posted comments on this change.
Change subject: Add per tablet write RPC throttling
......................................................................
Patch Set 5:
> Does the Java client also need a change to retry on THROTTLED error?
Currently THROTTLED is handled in server side and rewrite the 'error' to RPC
layer ERROR_SERVER_TOO_BUSY, so this doesn't require client side change.
// Generic "service unavailable" errors will cause the client to retry later.
if ((code == TabletServerErrorPB::UNKNOWN_ERROR ||
code == TabletServerErrorPB::THROTTLED) && s.IsServiceUnavailable()) {
context->RespondRpcFailure(rpc::ErrorStatusPB::ERROR_SERVER_TOO_BUSY, s);
return;
}
--
To view, visit http://gerrit.cloudera.org:8080/2327
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: comment
Gerrit-Change-Id: I33cb6934d27b883a783682cef1e0723100637d45
Gerrit-PatchSet: 5
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Binglin Chang <[email protected]>
Gerrit-Reviewer: Binglin Chang <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: No
