impala-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Kathy Sun (Code Review)" <>
Subject [Impala-ASF-CR] IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page
Date Wed, 17 Aug 2016 18:36:02 GMT
Kathy Sun has uploaded a new patch set (#5).

Change subject: IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

IMPALA-3981: Fix crash when accessing statestored / catalogd /memz page

The /memz page tried to add JVM metrics even when they didn't exist for
all daemons, not just Impala. This led to a crash when they tried to
access ExecEnv::GetInstance() without an initialised ExecEnv at
statestored and catalogd

To fix, changed the memz handler method to take an optional metric
group, provided by the caller.  memz handler will check the existence of
the metric group.

Used C++11 lambdas rather than boost::bind to help simplify the code.

Testing: Ran locally and looked at impalad/memz, statestored/memz
and catalogd/memz

Change-Id: If59b10f20044d1a468f27810a3029fe18fb19f29
M be/src/catalog/
M be/src/runtime/
M be/src/runtime/
M be/src/scheduling/
M be/src/service/
M be/src/statestore/
M be/src/statestore/
M be/src/util/
M be/src/util/default-path-handlers.h
M be/src/util/
M be/src/util/
M be/src/util/
M be/src/util/metrics.h
M www/memz.tmpl
14 files changed, 55 insertions(+), 31 deletions(-)

  git pull ssh:// refs/changes/98/3998/5
To view, visit
To unsubscribe, visit

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If59b10f20044d1a468f27810a3029fe18fb19f29
Gerrit-PatchSet: 5
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Kathy Sun <>
Gerrit-Reviewer: Henry Robinson <>
Gerrit-Reviewer: Kathy Sun <>
Gerrit-Reviewer: Tim Armstrong <>

View raw message