[kudu] branch master updated: [server] --rpc_default_keepalive_time_ms is not runtime

[al...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git


The following commit(s) were added to refs/heads/master by this push:
     new 13f5694  [server] --rpc_default_keepalive_time_ms is not runtime
13f5694 is described below

commit 13f5694e993284db7d8c85bbccc05db9e6f2eb03
Author: Alexey Serbin <al...@apache.org>
AuthorDate: Fri Sep 18 18:17:47 2020 -0700

    [server] --rpc_default_keepalive_time_ms is not runtime
    
    The --rpc_default_keepalive_time_ms flag is not runtime: the messenger
    for a Kudu server created in ServerBase::Init() using current value of
    the flag and the parameter cannot be changed later on.
    
    Change-Id: Id79bb22d7ce424a37b0a61b75c054906e7de0d1c
    Reviewed-on: http://gerrit.cloudera.org:8080/16476
    Tested-by: Kudu Jenkins
    Reviewed-by: Grant Henke <gr...@apache.org>
---
 src/kudu/server/server_base.cc | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/kudu/server/server_base.cc b/src/kudu/server/server_base.cc
index dcc8afd..3ccede8 100644
--- a/src/kudu/server/server_base.cc
+++ b/src/kudu/server/server_base.cc
@@ -103,7 +103,6 @@ TAG_FLAG(max_negotiation_threads, advanced);
 DEFINE_int64(rpc_negotiation_timeout_ms, 3000,
              "Timeout for negotiating an RPC connection.");
 TAG_FLAG(rpc_negotiation_timeout_ms, advanced);
-TAG_FLAG(rpc_negotiation_timeout_ms, runtime);
 
 DEFINE_bool(webserver_enabled, true, "Whether to enable the web server on this daemon. "
             "NOTE: disabling the web server is also likely to prevent monitoring systems "