Alexey Serbin has uploaded this change for review. ( 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) --- M src/kudu/clock/hybrid_clock-test.cc 1 file changed, 9 insertions(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/35/20335/1 -- 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: newchange Gerrit-Change-Id: Ie30ca383f50db1f62e140da5181e9a406367f7bd Gerrit-Change-Number: 20335 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
