Hello Tim Armstrong, Impala Public Jenkins,

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

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

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

Change subject: IMPALA-5031: link fesupport so FE tests run with UBSAN
......................................................................

IMPALA-5031: link fesupport so FE tests run with UBSAN

This commit enables the frontend and JDBC tests to run under
UBSAN. Before this comit, they fail with an error message about
finding the UBSAN support functions like
__ubsan_handle_load_invalid_value. Linking those functions into
libfesupport.so requires using the linker flag --whole-archive so that
these symbols, which are not directly referenced by the source code
used to build libfesupport.so, are included.

This patch also enables the custom cluster tests to run under UBSAN;
they recently gained frontend tests in commit
d72f3330c1edc9086ba120e6d3469a75c0aea083.

Change-Id: I42049fb3e2de83aee0d0e00e2703788afde739e2
---
M be/src/service/CMakeLists.txt
M bin/impala-config.sh
2 files changed, 13 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/10/13710/3
--
To view, visit http://gerrit.cloudera.org:8080/13710
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I42049fb3e2de83aee0d0e00e2703788afde739e2
Gerrit-Change-Number: 13710
Gerrit-PatchSet: 3
Gerrit-Owner: Jim Apple <jbap...@apache.org>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Jim Apple <jbap...@apache.org>
Gerrit-Reviewer: Tim Armstrong <tarmstr...@cloudera.com>

Reply via email to