Hello Jim Apple, Tim Armstrong, Joe McDonnell, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10404 to look at the new patch set (#4). Change subject: Moving default sanitizer options into init.cc from shell scripts. ...................................................................... Moving default sanitizer options into init.cc from shell scripts. When running tests with ASAN, you need to set ASAN_OPTIONS explicitly, to avoid various failures. In particular, backend tests fail complaining about memory leaks and tests that use the parquet-reader binary complain similarly. It turns out that we can shove the default options into our code base directly, avoiding the need for users to set it explicitly. I've done the same thing for TSAN and UBSAN. I've manually checked that these are being read. In the UBSAN case, I checked both with gdb and with inotifywatch on the suppressions file. Change-Id: I3cbbd210c67750a48003f336bea1f3e1cb2d9e6b --- M be/src/common/init.cc M bin/run-backend-tests.sh M bin/start-catalogd.sh M bin/start-impalad.sh M bin/start-statestored.sh 5 files changed, 27 insertions(+), 15 deletions(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/04/10404/4 -- To view, visit http://gerrit.cloudera.org:8080/10404 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newpatchset Gerrit-Change-Id: I3cbbd210c67750a48003f336bea1f3e1cb2d9e6b Gerrit-Change-Number: 10404 Gerrit-PatchSet: 4 Gerrit-Owner: Philip Zeyliger <phi...@cloudera.com> Gerrit-Reviewer: Jim Apple <jbapple-imp...@apache.org> Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com> Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com> Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>