Hello Dan Burkert, David Ribeiro Alves,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/7477

to review the following change.

Change subject: WIP: add a built-in NTP client
......................................................................

WIP: add a built-in NTP client

This adds a built-in stripped-down implementation of NTP without any
reliance on kernel features. This should hopefully make it easier for
users to configure NTP even if they don't have root, and also can
maintain better clock error than the system implementation, since we can
prioritize low error bounds rather than low jitter.

TODO: needs a lot more docs, testing, etc, plus the ability to configure
the NTP servers rather than hard-code them.

Change-Id: Ieb9eee9f0334236f39617492bd6f01304d1a0255
---
M CMakeLists.txt
A src/kudu/clock/CMakeLists.txt
A src/kudu/clock/builtin_ntp.cc
A src/kudu/clock/builtin_ntp.h
A src/kudu/clock/mock_ntp.cc
A src/kudu/clock/mock_ntp.h
A src/kudu/clock/ntp_source.h
A src/kudu/clock/system_ntp.cc
A src/kudu/clock/system_ntp.h
M src/kudu/integration-tests/consistency-itest.cc
M src/kudu/integration-tests/tablet_history_gc-itest.cc
M src/kudu/server/CMakeLists.txt
M src/kudu/server/generic_service.cc
M src/kudu/server/hybrid_clock-test.cc
M src/kudu/server/hybrid_clock.cc
M src/kudu/server/hybrid_clock.h
M src/kudu/server/server_base.proto
M src/kudu/tablet/tablet_history_gc-test.cc
18 files changed, 1,208 insertions(+), 177 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/77/7477/1
-- 
To view, visit http://gerrit.cloudera.org:8080/7477
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ieb9eee9f0334236f39617492bd6f01304d1a0255
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com>

Reply via email to