impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Impala Public Jenkins (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-5756: start memory maintenance thread after metric creation
Date Thu, 03 Aug 2017 17:28:32 GMT
Impala Public Jenkins has submitted this change and it was merged.

Change subject: IMPALA-5756: start memory maintenance thread after metric creation
......................................................................


IMPALA-5756: start memory maintenance thread after metric creation

Daemons were sometimes crashing on startup if the memory maintenance
thread started running before the memory metrics were created.

This changes the startup sequence so that the thread is started
after the metrics are registered.

Change-Id: Ibb7337a68547b4d660f3100b04df2c20befb044c
Reviewed-on: http://gerrit.cloudera.org:8080/7573
Reviewed-by: Henry Robinson <henry@cloudera.com>
Tested-by: Impala Public Jenkins
---
M be/src/catalog/catalogd-main.cc
M be/src/common/init.cc
M be/src/common/init.h
M be/src/service/impalad-main.cc
M be/src/statestore/statestored-main.cc
5 files changed, 13 insertions(+), 8 deletions(-)

Approvals:
  Impala Public Jenkins: Verified
  Henry Robinson: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ibb7337a68547b4d660f3100b04df2c20befb044c
Gerrit-PatchSet: 3
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <tarmstrong@cloudera.com>
Gerrit-Reviewer: Henry Robinson <henry@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins
Gerrit-Reviewer: Tim Armstrong <tarmstrong@cloudera.com>

Mime
View raw message