Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14417 )
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 <[email protected]> (cherry picked from commit 2768c156186e3da9693ddeff78a8d3206827fb88) Reviewed-on: http://gerrit.cloudera.org:8080/14417 Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/clock/builtin_ntp.cc 1 file changed, 8 insertions(+), 12 deletions(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I77ba6ea37b0e94058eaab018ae4d66976f0482cd Gerrit-Change-Number: 14417 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
