Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15209 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/15209 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/clock/builtin_ntp.cc M src/kudu/mini-cluster/external_mini_cluster-test.cc 2 files changed, 14 insertions(+), 13 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- 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: merged Gerrit-Change-Id: Iaa419013f047d8ec7803092f8225199e9e6f985b Gerrit-Change-Number: 15209 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
