Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20335 )
Change subject: [clock] fix HybridClockTest.TimeSourceAutoSelection ...................................................................... [clock] fix HybridClockTest.TimeSourceAutoSelection It was reported that HybridClockTest.TimeSourceAutoSelection scenario had failed with the following message: src/kudu/clock/hybrid_clock-test.cc:501: Failure Value of: s.ToString() Expected: has substring "Error reading clock. Clock considered unsynchronized" Actual: "Service unavailable: timed out waiting for clock synchronisation: wallclock is not synchronized: no valid NTP responses yet" (of type std::string) This patch addresses the issue, updating the pattern of the expected error message to account for the string output by the built-in NTP clock as well. Change-Id: Ie30ca383f50db1f62e140da5181e9a406367f7bd Reviewed-on: http://gerrit.cloudera.org:8080/20324 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Mahesh Reddy <[email protected]> Reviewed-by: Yifan Zhang <[email protected]> (cherry picked from commit ebf0a645d5deb4be40faf441e230be07016fb683) Reviewed-on: http://gerrit.cloudera.org:8080/20335 Tested-by: Yingchun Lai <[email protected]> --- M src/kudu/clock/hybrid_clock-test.cc 1 file changed, 9 insertions(+), 2 deletions(-) Approvals: Yifan Zhang: Looks good to me, approved Yingchun Lai: Verified -- To view, visit http://gerrit.cloudera.org:8080/20335 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.17.x Gerrit-MessageType: merged Gerrit-Change-Id: Ie30ca383f50db1f62e140da5181e9a406367f7bd Gerrit-Change-Number: 20335 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yifan Zhang <[email protected]> Gerrit-Reviewer: Yingchun Lai <[email protected]>
