Skip to content

Commit

Permalink
[#21963] YSQL: Add time_source to gflag_allowlist.txt
Browse files Browse the repository at this point in the history
Summary:
time_source does not have any secrets. Call home info on time_source is useful.

Also, time_source is a non-runtime flag.
Jira: DB-10879

Test Plan:
Jenkins

Backport-through: 2024.2

Reviewers: hsunder, smishra

Reviewed By: hsunder

Subscribers: ybase

Differential Revision: https://phorge.dev.yugabyte.com/D39031
  • Loading branch information
pao214 committed Oct 23, 2024
1 parent 21d7ad3 commit d5c096f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions src/yb/gflag_allowlist.txt
Original file line number Diff line number Diff line change
Expand Up @@ -41,6 +41,7 @@ ref_counted_debug_type_name_regex
regular_tablets_data_block_key_value_encoding
rocksdb_compact_flush_rate_limit_sharing_mode
ssl_protocols
time_source
trace_to_console
tryfromenv
undefok
Expand Down
2 changes: 1 addition & 1 deletion src/yb/server/hybrid_clock.cc
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ METRIC_DEFINE_gauge_int64(server, hybrid_clock_skew,
yb::MetricUnit::kMicroseconds,
"Server clock skew.");

DEFINE_UNKNOWN_string(time_source, "",
DEFINE_NON_RUNTIME_string(time_source, "",
"The clock source that HybridClock should use. "
"Leave empty for WallClock, clockbound for ClockboundClock, "
"and other values depend on added clock providers and "
Expand Down

0 comments on commit d5c096f

Please sign in to comment.