Alexey Serbin has uploaded this change for review. (
http://gerrit.cloudera.org:8080/14908
Change subject: [master/tserver] return non-zero status from main() instead of
crashing
......................................................................
[master/tserver] return non-zero status from main() instead of crashing
Prior to this patch, Kudu masters and tablet servers would crash if
{Master,TabletServer}::{Init,Start}() returned non-OK status. As it's
seen, there is not much advantage in that behavior when in the context
of main() function compared with returning non-zero code from main():
* From the monitoring and reporting perspectives, it's possible to
distinguish between non-OK runs based on the exit status of
the process as well.
* In most cases, core dumps were disabled, and only minidumps were
available from processes crashed in such cases. However, given
a minidump, there wasn't much information available for
troubleshooting because of the stripped heap.
This patch changes the described behavior, so now Kudu masters and
tablet servers write an error message into the log and exit with
non-zero status instead of crashing.
Change-Id: Id06646e2211eb24db28c582455d4a34af7501b26
---
M src/kudu/master/master_main.cc
M src/kudu/tserver/tablet_server_main.cc
M src/kudu/util/logging.h
3 files changed, 20 insertions(+), 15 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/08/14908/1
--
To view, visit http://gerrit.cloudera.org:8080/14908
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Id06646e2211eb24db28c582455d4a34af7501b26
Gerrit-Change-Number: 14908
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
