Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17582
to look at the new patch set (#2).
Change subject: WIP [clock] change clock source selection for 'auto'
......................................................................
WIP [clock] change clock source selection for 'auto'
This patch changes the logic to select particular time source the
pseudo-source 'auto' used for --time_source.
Before this patch, the 'system' time source would be selected
if a Kudu server --time_source=auto is run on a non-cloud VM
or other environment where the cloud instance detector isn't
aware of a dedicated NTP server (which is usually available
via the host-only interface).
After this patch, the 'builtin' time source would be selected
if a Kudu server runs with --time_source=auto on a non-cloud VM
or other environment where the cloud instance detector isn't
aware of dedicated NTP servers AND the --builtin_ntp_servers flag
is set to a valid value.
WIP:
* collect initial feedback
* update/add tests to cover the newly introduced functionality
Change-Id: I72cb54c488b2aa2c73acd6e6e5f6e50dd5811175
---
M src/kudu/clock/builtin_ntp.cc
M src/kudu/clock/builtin_ntp.h
M src/kudu/clock/hybrid_clock.cc
M src/kudu/clock/hybrid_clock.h
M src/kudu/clock/ntp-test.cc
M src/kudu/server/default_path_handlers.cc
6 files changed, 83 insertions(+), 54 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/82/17582/2
--
To view, visit http://gerrit.cloudera.org:8080/17582
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I72cb54c488b2aa2c73acd6e6e5f6e50dd5811175
Gerrit-Change-Number: 17582
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
