Impala Public Jenkins has submitted this change and it was merged. (
http://gerrit.cloudera.org:8080/10895 )
Change subject: IMPALA-7129: Fixing path to UBSAN suppressions.
......................................................................
IMPALA-7129: Fixing path to UBSAN suppressions.
This pins the suppression file path at build time.
It turns out that calling getenv("IMPALA_HOME") was returning null at
the time the undefined behavior was being initialized. I suspect this is
an initialization ordering issue. To step around the issue, I'm simply
using a preprocessor macro to get at the suppression file path.
I was able to reproduce the error by running the catalogd binary
outside of $IMPALA_HOME. Before this change, that would fail reliably.
Change-Id: I5295c2d1bddbea585a761b85a51eadecf10d191d
Reviewed-on: http://gerrit.cloudera.org:8080/10895
Reviewed-by: Jim Apple <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
---
M be/CMakeLists.txt
M be/src/common/init.cc
2 files changed, 8 insertions(+), 5 deletions(-)
Approvals:
Jim Apple: Looks good to me, approved
Impala Public Jenkins: Verified
--
To view, visit http://gerrit.cloudera.org:8080/10895
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I5295c2d1bddbea585a761b85a51eadecf10d191d
Gerrit-Change-Number: 10895
Gerrit-PatchSet: 7
Gerrit-Owner: Philip Zeyliger <[email protected]>
Gerrit-Reviewer: Fredy Wijaya <[email protected]>
Gerrit-Reviewer: Impala Public Jenkins <[email protected]>
Gerrit-Reviewer: Jim Apple <[email protected]>
Gerrit-Reviewer: Philip Zeyliger <[email protected]>
