Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17582
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/hybrid_clock.h M src/kudu/clock/ntp-test.cc 3 files changed, 28 insertions(+), 9 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/82/17582/1 -- 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: newchange Gerrit-Change-Id: I72cb54c488b2aa2c73acd6e6e5f6e50dd5811175 Gerrit-Change-Number: 17582 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
