Tim Armstrong has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/11871


Change subject: IMPALA-6436: exit instead of abort for catalog startup failure
......................................................................

IMPALA-6436: exit instead of abort for catalog startup failure

Rename EXIT_WITH_EXC to ABORT_WITH_EXC to make the behaviour more
obvious at callsites.

Handle exceptions from Catalog constructor by logging the backtrace and
exiting cleanly, rather than aborting. This will prevent generation of a
coredump or minidump.

Testing:
Tested starting the catalogd locally without the HMS running and a
low connection timeout:

  start-impala-cluster.py --catalogd_args=--initial_hms_cnxn_timeout_s=2

Confirmed that the backtrace was logged to catalogd.ERROR and that no
core or minidump was generated.

Change-Id: I4026dccb39843b847426112fc0fe9ba897e48dcc
---
M be/src/catalog/catalog.cc
M be/src/scheduling/request-pool-service.cc
M be/src/service/fe-support.cc
M be/src/service/frontend.cc
M be/src/util/jni-util.h
M be/src/util/logging-support.cc
M be/src/util/zip-util.cc
7 files changed, 40 insertions(+), 21 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/71/11871/1
--
To view, visit http://gerrit.cloudera.org:8080/11871
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I4026dccb39843b847426112fc0fe9ba897e48dcc
Gerrit-Change-Number: 11871
Gerrit-PatchSet: 1
Gerrit-Owner: Tim Armstrong <[email protected]>

Reply via email to