Todd Lipcon has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/9471 )

Change subject: debug-util-test: add status message when stacks fail
......................................................................

debug-util-test: add status message when stacks fail

The TestSnapshot test case is flaky, and it seems to be because one of the
threads sometimes fails to take a stack trace. This patch just adds the Status
to the log so that we can diagnose the flake next time it happens.

Change-Id: I4f1da4658c2fbb8e4a049edc392b15f7bf14c8fa
Reviewed-on: http://gerrit.cloudera.org:8080/9471
Reviewed-by: Adar Dembo <a...@cloudera.com>
Tested-by: Kudu Jenkins
---
M src/kudu/util/debug-util-test.cc
1 file changed, 3 insertions(+), 2 deletions(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified

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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I4f1da4658c2fbb8e4a049edc392b15f7bf14c8fa
Gerrit-Change-Number: 9471
Gerrit-PatchSet: 3
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to