Todd Lipcon has submitted this change and it was merged. Change subject: tablet_peer: update status message on failure to start ......................................................................
tablet_peer: update status message on failure to start If the tablet peer fails to start up, we were calling SetFailed(), but this didn't actually update the status message which would later be reported as part of the TabletStatusPB. This made for confusing debug experiences. This now surfaces the error. Change-Id: I6b6e53a33fde296d99be7027dbe75ac057920c20 Reviewed-on: http://gerrit.cloudera.org:8080/3682 Tested-by: Kudu Jenkins Reviewed-by: Jean-Daniel Cryans <jdcry...@apache.org> --- M src/kudu/tablet/tablet_peer.cc M src/kudu/tablet/tablet_peer.h M src/kudu/tserver/tablet_server-test-base.h M src/kudu/tserver/tablet_server-test.cc 4 files changed, 22 insertions(+), 8 deletions(-) Approvals: Jean-Daniel Cryans: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/3682 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6b6e53a33fde296d99be7027dbe75ac057920c20 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
