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);