impala-reviews mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Lars Volker (Code Review)" <ger...@cloudera.org>
Subject [native-toolchain-CR] IMPALA-5208, IMPALA-5187: Fixes for Breakpad #681, #728
Date Thu, 11 May 2017 21:28:38 GMT
Lars Volker has uploaded a new change for review.

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

Change subject: IMPALA-5208, IMPALA-5187: Fixes for Breakpad #681, #728
......................................................................

IMPALA-5208, IMPALA-5187: Fixes for Breakpad #681, #728

IMPALA-5208 depends on Breakpad #681:
Breakpad had an issue [1] with random GUID generation on Linux, that
lead to processes started within the same second to write minidumps to
the same filename. Only one would win and overwrite minidumps of the
other processes.

IMPALA-5187 depends on Breakpad #728:
When writing a minidump on Linux, Breakpad called clone() in
linux/handler/exception_handler.cc with the CLONE_FILES flag. If the
parent process died while the child waited for the continuation signal,
the write side of the pipe 'fdes' stayed open in the child. The child
would not receive a SIGPIPE and would wait forever.

This change bumps the Breakpad version in the toolchain to include fixes
for the upstream bugs. It will allow us to remove the workaround we
introduced to address IMPALA-3794. I tested this by reverting the
workaround, removing any startup delay from the cluster startup scripts,
and then verified that the test_breakpad.py passes.

This change also removes the patch to increase the maximum number of
threads for minidump_stackwalk, which has been merged upstream.

[1] https://bugs.chromium.org/p/google-breakpad/issues/detail?id=681

Change-Id: If080919f0d241eac90edd8d23bf6007e09157e23
---
M buildall.sh
A source/breakpad/breakpad-ffe3e478657dc7126fca6329dfcedc49f4c726d9-patches/0001-Add-basic-support-for-dwz-dwarf-extension.patch
2 files changed, 78 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/native-toolchain refs/changes/66/6866/1
-- 
To view, visit http://gerrit.cloudera.org:8080/6866
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If080919f0d241eac90edd8d23bf6007e09157e23
Gerrit-PatchSet: 1
Gerrit-Project: native-toolchain
Gerrit-Branch: master
Gerrit-Owner: Lars Volker <lv@cloudera.com>

Mime
View raw message