Hello Kudu Jenkins, Grant Henke,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/14417
to review the following change.
Change subject: [clock] log on CombineClocks() status only via VLOG
......................................................................
[clock] log on CombineClocks() status only via VLOG
The logs about the walltime-would-move-back outcome of
BuiltInNtp::CombineClocks() look a bit scary, but de facto they
are benign and of nothing irregular in case of jittery reference
servers. Let's output them as VLOG(1) messages. These should
eventually be gone once the proper clock selection algorithm is
implemented (see KUDU-2939 for details).
Change-Id: I77ba6ea37b0e94058eaab018ae4d66976f0482cd
Reviewed-on: http://gerrit.cloudera.org:8080/14416
Tested-by: Kudu Jenkins
Reviewed-by: Grant Henke <granthe...@apache.org>
(cherry picked from commit 2768c156186e3da9693ddeff78a8d3206827fb88)
---
M src/kudu/clock/builtin_ntp.cc
1 file changed, 8 insertions(+), 12 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/17/14417/1
--
To view, visit http://gerrit.cloudera.org:8080/14417
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: branch-1.11.x
Gerrit-MessageType: newchange
Gerrit-Change-Id: I77ba6ea37b0e94058eaab018ae4d66976f0482cd
Gerrit-Change-Number: 14417
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <aser...@cloudera.com>
Gerrit-Reviewer: Grant Henke <granthe...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
