Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10651 )
Change subject: Increase --tablet_copy_begin_session_timeout_ms to 30 sec ...................................................................... Increase --tablet_copy_begin_session_timeout_ms to 30 sec This patch bumps the tablet copy begin session timeout from 3 seconds to 30 seconds. 3 seconds is too low of a value in cases where there is a lot of disk activity on the remote server. This patch also tweaks a couple of other minor things, like explicitly setting the controller timeout in the case of a StartTabletCopy() RPC, however there is no functionality change there because the same RpcController is shared between UpdateConsensus() and StartTabletCopy() calls, and the same timeout value is used for both. I just found it confusing that the timeout was explicitly set for one but not the other. Change-Id: I9e364d60351ad925583f8d7ef9dea55e0dac2507 Reviewed-on: http://gerrit.cloudera.org:8080/10651 Reviewed-by: Will Berkeley <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/consensus/consensus_peers.cc M src/kudu/tserver/tablet_copy_client.cc 2 files changed, 5 insertions(+), 3 deletions(-) Approvals: Will Berkeley: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/10651 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I9e364d60351ad925583f8d7ef9dea55e0dac2507 Gerrit-Change-Number: 10651 Gerrit-PatchSet: 3 Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]>
