Joe McDonnell has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/18129


Change subject: IMPALA-10951 (preparation): Update Kudu to a more recent version
......................................................................

IMPALA-10951 (preparation): Update Kudu to a more recent version

As part of moving to a newer protobuf, this updates the Kudu version
to get the fix for KUDU-3334. With this newer Kudu version, Clang
builds hit an error while linking:
lib/libLLVMCodeGen.a(TargetPassConfig.cpp.o):TargetPassConfig.cpp:function 
llvm::TargetPassConfig::createRegAllocPass(bool): error: relocation refers to 
global symbol "std::call_once<void (&)()>(std::once_flag&, void 
(&)())::{lambda()#2}::_FUN()", which is defined in a discarded section
  section group signature: 
"_ZZSt9call_onceIRFvvEJEEvRSt9once_flagOT_DpOT0_ENKUlvE0_clEv"
  prevailing definition is from 
../../build/debug/security/libsecurity.a(openssl_util.cc.o)
(This is from a newer binutils that will be pursued separately.)

As a hack to get around this error, this adds the calloncehack
shared library. The shared library publicly defines the symbol that
was coming from kudu_client. By linking it ahead of kudu_client, the
linker uses that rather than the one from kudu_client. This fixes
the Clang builds.

The new Kudu also requires a minor change to the flags for tserver
startup.

Testing:
 - Ran debug tests and verified calloncehack is not used
 - Ran ASAN tests

Change-Id: Ieccbe284f11445e1de792352ebc7c9e1fa2ca0c3
---
M CMakeLists.txt
M be/CMakeLists.txt
M be/src/service/impala-server.cc
M bin/bootstrap_toolchain.py
M bin/impala-config.sh
A cmake_modules/FindCallOnceHack.cmake
M testdata/cluster/node_templates/common/etc/kudu/tserver.conf.tmpl
7 files changed, 91 insertions(+), 6 deletions(-)



  git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/29/18129/1
--
To view, visit http://gerrit.cloudera.org:8080/18129
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ieccbe284f11445e1de792352ebc7c9e1fa2ca0c3
Gerrit-Change-Number: 18129
Gerrit-PatchSet: 1
Gerrit-Owner: Joe McDonnell <joemcdonn...@cloudera.com>

Reply via email to