Hello Attila Bukor, Kudu Jenkins, Abhishek Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/20313
to look at the new patch set (#3).
Change subject: [test_util] more accurate status codes for WaitForBind()
......................................................................
[test_util] more accurate status codes for WaitForBind()
When troubleshooting failed test scenarios involving the mini_ranger and
the mini_ranger_kms test wrappers, I found it's hard to decipher error
messages when the underlying memory hogs were slow to start within the
specified timeout of 90 seconds.
This patch addresses the issue, so in case of timeout the WaitForBind()
utility returns Status::TimedOut() with comprehensive error message.
I also took the liberty of making the timeout tracking more precise and
fixing minor code style issues in the related code.
Change-Id: Ic4ed06540d3f4d86bb61f96ffb36ffc671e1c485
---
M src/kudu/ranger-kms/mini_ranger_kms.cc
M src/kudu/ranger/mini_ranger.cc
M src/kudu/util/test_util.cc
3 files changed, 30 insertions(+), 18 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/13/20313/3
--
To view, visit http://gerrit.cloudera.org:8080/20313
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic4ed06540d3f4d86bb61f96ffb36ffc671e1c485
Gerrit-Change-Number: 20313
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
