Abhishek Chennaka has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/20337 )

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]>
---
M src/kudu/clock/hybrid_clock.cc
1 file changed, 5 insertions(+), 3 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Abhishek Chennaka: Looks good to me, approved

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib1bdd13cd2894954002a79258a46aeaee1a44fcf
Gerrit-Change-Number: 20337
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to