Hello Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/15209
to look at the new patch set (#2).
Change subject: [clock] small cleanup on built-in NTP client flags
......................................................................
[clock] small cleanup on built-in NTP client flags
This patch renames --builtin_ntp_client_enable_auto_config_in_cloud
flag into --builtin_ntp_client_enable_auto_config and updates the
flag's description to contain more details on the system behavior
behind the flag. I also updated the warning message output upon
falling back to the specified set of NTP servers when automatic
discovery of NTP servers yields no result.
The rationale for this change is to shorten the name of the flag
and be able to use the flag in future for the auto-discovery of
NTP servers provided by other means (e.g., DHCP, etc.).
Change-Id: Iaa419013f047d8ec7803092f8225199e9e6f985b
---
M src/kudu/clock/builtin_ntp.cc
M src/kudu/mini-cluster/external_mini_cluster-test.cc
2 files changed, 14 insertions(+), 13 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/09/15209/2
--
To view, visit http://gerrit.cloudera.org:8080/15209
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iaa419013f047d8ec7803092f8225199e9e6f985b
Gerrit-Change-Number: 15209
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)