You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2020/06/04 01:41:11 UTC
[kudu] 01/03: [tserver] add 'runtime' tag for scanner_ttl_ms
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
commit 3492cd818768b03baab411d90356e7fa21e58cf5
Author: zhangyifan27 <ch...@163.com>
AuthorDate: Wed Jun 3 16:51:59 2020 +0800
[tserver] add 'runtime' tag for scanner_ttl_ms
The --scanner_ttl_ms flag has runtime-settable behavior,
this patch simply adds 'runtime' tag so we don't need
to use '--force' when using 'kudu tserver set_flag'.
Change-Id: I91c2a9192f7f80edc3927fe226ddcd1d7be0d78f
Reviewed-on: http://gerrit.cloudera.org:8080/16028
Tested-by: Kudu Jenkins
Reviewed-by: Yingchun Lai <40...@qq.com>
Reviewed-by: Todd Lipcon <to...@apache.org>
---
src/kudu/tserver/scanners.cc | 2 ++
1 file changed, 2 insertions(+)
diff --git a/src/kudu/tserver/scanners.cc b/src/kudu/tserver/scanners.cc
index 41ea594..add5d9d 100644
--- a/src/kudu/tserver/scanners.cc
+++ b/src/kudu/tserver/scanners.cc
@@ -53,6 +53,8 @@ DEFINE_int32(scanner_ttl_ms, 60000,
"Number of milliseconds of inactivity allowed for a scanner"
"before it may be expired");
TAG_FLAG(scanner_ttl_ms, advanced);
+TAG_FLAG(scanner_ttl_ms, runtime);
+
DEFINE_int32(scanner_gc_check_interval_us, 5 * 1000L *1000L, // 5 seconds
"Number of microseconds in the interval at which we remove expired scanners");
TAG_FLAG(scanner_gc_check_interval_us, hidden);