Hello Todd Lipcon,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/4006

to review the following change.

Change subject: mini_cluster: avoid SIGSEGV when master fails to start up
......................................................................

mini_cluster: avoid SIGSEGV when master fails to start up

My machine's clock became desynchronized causing MiniCluster-based tests to
SIGSEGV. The root cause appears to be in how we manage the mini_masters_
vector. By resizing it right at the beginning, various code paths later on
may iterate on some number of non-existent masters and dereference empty
shared pointers.

Change-Id: Id1cc262ba83f70d97434fca4c75f76b095db77d6
---
M src/kudu/integration-tests/mini_cluster.cc
1 file changed, 5 insertions(+), 9 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/06/4006/1
-- 
To view, visit http://gerrit.cloudera.org:8080/4006
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Id1cc262ba83f70d97434fca4c75f76b095db77d6
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to