impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Tim Armstrong (Code Review)" <ger...@cloudera.org>
Subject [Impala-ASF-CR] IMPALA-5230: fix non-functional impalad under ASAN
Date Wed, 19 Apr 2017 22:15:29 GMT
Tim Armstrong has uploaded a new change for review.

  http://gerrit.cloudera.org:8080/6694

Change subject: IMPALA-5230: fix non-functional impalad under ASAN
......................................................................

IMPALA-5230: fix non-functional impalad under ASAN

The bug is that TCMalloc metrics are not initialised under
ASAN but other metrics that reference them are created. As
soon as a debug webpage tries to access that metric, Impala
crashes.

The bug was introduced by commit 955b257cfb5: "IMPALA-5073:
Part 1: add option to use mmap() for buffer pool".

Testing:
Started the cluster under ASAN, ran some query tests, checked the /memz
and /metrics pages.

Change-Id: Ic71fa57e0444f470398e16513e4ba6e7f71565e4
---
M be/src/util/memory-metrics.cc
M be/src/util/memory-metrics.h
2 files changed, 2 insertions(+), 2 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic71fa57e0444f470398e16513e4ba6e7f71565e4
Gerrit-PatchSet: 1
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-Owner: Tim Armstrong <tarmstrong@cloudera.com>

Mime
View raw message