Alexey Serbin has uploaded a new patch set (#10). Change subject: KUDU-2027 retry scan RPC if negotiation times out ......................................................................
KUDU-2027 retry scan RPC if negotiation times out This patch addresses KUDU-2027, i.e. with this patch the Kudu C++ client retries a scan RPC with other tablet replica if the connection negotiation with current replica timed out. Also addressed a TODO in KuduScanner::Data::OpenTablet(). As a part of the fix for KUDU-2027, I updated the logic of the timeout handling. Now, regardless of whether it was the RPC or the overall operation timeout, the corresponding tablet server is marked as failed and blacklisted. Added new integration test to cover the updated client's behavior. This patch is a follow-up for e3c5dd18c22b9e20358f05dcd301e7736c0a3321, since KUDU-2027 is a scan-related counterpart of KUDU-1580. Change-Id: I05a3c38cc07c9b6edd1ae773d9850e92645f3b14 --- M src/kudu/client/client-test.cc M src/kudu/client/client.cc M src/kudu/client/scanner-internal.cc M src/kudu/client/scanner-internal.h M src/kudu/integration-tests/client-negotiation-failover-itest.cc 5 files changed, 269 insertions(+), 103 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/37/7037/10 -- To view, visit http://gerrit.cloudera.org:8080/7037 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I05a3c38cc07c9b6edd1ae773d9850e92645f3b14 Gerrit-PatchSet: 10 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[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]>
