Hello Taras Bobrovytsky, Alex Behm, Dan Hecht,

I'd like you to reexamine a change.  Please visit

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

to look at the new patch set (#4).

Change subject: IMPALA-3581: Change location of minidump folders to log_dir
......................................................................

IMPALA-3581: Change location of minidump folders to log_dir

Currently the default minidump location is /tmp/impala-minidumps, which can be 
wiped on
reboot on various distributions. This change moves the default location to
FLAGS_log_dir/minidumps/$daemon. The additional trailing $daemon folder is kept 
to prevent
name collisions in case of local test clusters and strangely configured 
installations.

For local test clusters the minidumps will be written to
$IMPALA_HOME/logs/cluster/minidumps/{catalogd,impalad,statestored}.

Change-Id: Idecf5a314bfb8b0870e8aa4819c4fb39a107702f
---
M be/src/common/global-flags.cc
M be/src/util/minidump.cc
M bin/collect_minidumps.py
M bin/generate_minidump_collection_testdata.py
M tests/custom_cluster/test_breakpad.py
5 files changed, 63 insertions(+), 27 deletions(-)


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

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Idecf5a314bfb8b0870e8aa4819c4fb39a107702f
Gerrit-PatchSet: 4
Gerrit-Project: Impala
Gerrit-Branch: cdh5-2.6.0_5.8.0
Gerrit-Owner: Lars Volker <[email protected]>
Gerrit-Reviewer: Alex Behm <[email protected]>
Gerrit-Reviewer: Dan Hecht <[email protected]>
Gerrit-Reviewer: Lars Volker <[email protected]>
Gerrit-Reviewer: Taras Bobrovytsky <[email protected]>

Reply via email to