This is an automated email from the ASF dual-hosted git repository.
zhangyifan 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 c5f358038 [tablet] --tablet_history_max_age_sec is a runtime flag
c5f358038 is described below
commit c5f358038e583577a91db7c16b488fe07503ad75
Author: Alexey Serbin <[email protected]>
AuthorDate: Tue Nov 29 13:31:32 2022 -0800
[tablet] --tablet_history_max_age_sec is a runtime flag
The way how --tablet_history_max_age_sec is used in
Tablet::GetTabletAncientHistoryMark() mandates a 'runtime' tag for the
flag. That allows for using the `kudu tserver set_flag` CLI tool
without adding the --force option when updating the setting.
Change-Id: Ie2b1553325cd33953d104b5bb3e8aeb73d90642a
Reviewed-on: http://gerrit.cloudera.org:8080/19286
Tested-by: Alexey Serbin <[email protected]>
Reviewed-by: Yifan Zhang <[email protected]>
---
src/kudu/tablet/tablet.cc | 1 +
1 file changed, 1 insertion(+)
diff --git a/src/kudu/tablet/tablet.cc b/src/kudu/tablet/tablet.cc
index e60443ff5..18ca96089 100644
--- a/src/kudu/tablet/tablet.cc
+++ b/src/kudu/tablet/tablet.cc
@@ -154,6 +154,7 @@ DEFINE_int32(tablet_history_max_age_sec, 60 * 60 * 24 * 7,
"initiated at a snapshot that is older than this age will be "
"rejected. To disable history removal, set to -1.");
TAG_FLAG(tablet_history_max_age_sec, advanced);
+TAG_FLAG(tablet_history_max_age_sec, runtime);
TAG_FLAG(tablet_history_max_age_sec, stable);
// Large encoded keys cause problems because we store the min/max encoded key
in the
