Repository: kudu
Updated Branches:
  refs/heads/master 7bcfcf64c -> 1746ae021


[c++ client] use default RPC timeout for batchers

If not overridden by KuduSession::SetTimeoutMillis(), use per-client
default RPC timeout (KuduClient::default_rpc_timeout()) for RPC
operations initiated by batchers.  Previously, the hard-coded setting
of 60 seconds was used instead.

Change-Id: Ic44acd5e2723295e115677f83e3fcf4074a61e52
Reviewed-on: http://gerrit.cloudera.org:8080/4703
Reviewed-by: Adar Dembo <a...@cloudera.com>
Tested-by: Kudu Jenkins


Project: http://git-wip-us.apache.org/repos/asf/kudu/repo
Commit: http://git-wip-us.apache.org/repos/asf/kudu/commit/5195ce57
Tree: http://git-wip-us.apache.org/repos/asf/kudu/tree/5195ce57
Diff: http://git-wip-us.apache.org/repos/asf/kudu/diff/5195ce57

Branch: refs/heads/master
Commit: 5195ce573850653e0e53094cdd35a1da93d33444
Parents: 7bcfcf6
Author: Alexey Serbin <aser...@cloudera.com>
Authored: Wed Oct 12 15:14:51 2016 -0700
Committer: Alexey Serbin <aser...@cloudera.com>
Committed: Thu Oct 13 00:18:09 2016 +0000

----------------------------------------------------------------------
 src/kudu/client/batcher.cc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kudu/blob/5195ce57/src/kudu/client/batcher.cc
----------------------------------------------------------------------
diff --git a/src/kudu/client/batcher.cc b/src/kudu/client/batcher.cc
index d7c4dca..9bb36dc 100644
--- a/src/kudu/client/batcher.cc
+++ b/src/kudu/client/batcher.cc
@@ -389,7 +389,7 @@ Batcher::Batcher(KuduClient* client,
     had_errors_(false),
     flush_callback_(nullptr),
     next_op_sequence_number_(0),
-    timeout_(MonoDelta::FromSeconds(60)),
+    timeout_(client->default_rpc_timeout()),
     outstanding_lookups_(0),
     buffer_bytes_used_(0) {
 }

Reply via email to