Yingchun Lai has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/20408 )

Change subject: [clock] clearer logging on the hybrid clock time source
......................................................................

[clock] clearer logging on the hybrid clock time source

This patch updates the logging messages about the time source used
for the hybrid clock's initialization.

Prior to this patch, it could be reported that the system auto-selected
a particular time source, but in fact the effective time source would be
specified explicitly via the --time_source flag and it was not 'auto',
so the time source auto-selection hadn't been involved at all.  That's
confusing, so here is a fix.

Change-Id: Ib1bdd13cd2894954002a79258a46aeaee1a44fcf
Reviewed-on: http://gerrit.cloudera.org:8080/20337
Tested-by: Kudu Jenkins
Reviewed-by: Abhishek Chennaka <[email protected]>
(cherry picked from commit c96a8377038e4dea7ebafb2afe6afe1fb8da245d)
Reviewed-on: http://gerrit.cloudera.org:8080/20408
Reviewed-by: Yingchun Lai <[email protected]>
---
M src/kudu/clock/hybrid_clock.cc
1 file changed, 5 insertions(+), 3 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Yingchun Lai: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/20408
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: branch-1.17.x
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib1bdd13cd2894954002a79258a46aeaee1a44fcf
Gerrit-Change-Number: 20408
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yingchun Lai <[email protected]>

Reply via email to