Alexey Serbin has submitted this change and it was merged. Change subject: KUDU-1034 client does not failover due to timeout ......................................................................
KUDU-1034 client does not failover due to timeout This patch fixes the issue described by KUDU-1034: the client does not mark the failed tablet server as 'failed' in case of timeout and continues to use it over and over again to send further requests, even if other tablet replicas might be available. Besides the actual fix, this patch incorporates an integration test (ClientFailoverTServerTimeoutITest.FailoverOnLeaderTimeout) written by Mike. Change-Id: Icfcece485e4053d921ffdc865612b3e7b9a992a3 Reviewed-on: http://gerrit.cloudera.org:8080/6924 Reviewed-by: Mike Percy <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/client/meta_cache.h M src/kudu/integration-tests/client_failover-itest.cc M src/kudu/integration-tests/tablet_copy-itest.cc M src/kudu/rpc/retriable_rpc.h 4 files changed, 125 insertions(+), 3 deletions(-) Approvals: Mike Percy: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/6924 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Icfcece485e4053d921ffdc865612b3e7b9a992a3 Gerrit-PatchSet: 8 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]>
