Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/20545 )

Change subject: [thirpdarty] upgrade gperftools to 2.13
......................................................................

[thirpdarty] upgrade gperftools to 2.13

With gperftools 2.13 come many updates since 2.8.1.  At least,
as a cumulative update from 2.11rc2 (a.k.a. 2.10.80 pre-release)
comes the support for generic_fp stack collection method on aarch64
platform [1], so now there is a parity between x86_64 and aarch64 builds
in terms of available options for stack trace collection in gperftools.

Also, the libgcc-based stacktrace capturing is more robust with the new
version of gperftools.  In addition, version 2.13 includes many other
fixes since 2.8.1, for details see [2].

With the new version, by default gperftools are built with the
'-fno-omit-frame-pointer -momit-leaf-frame-pointer' flag combination
if supported by the platform, so I removed the --enable-frame-pointers
flag for the configure script.

The gperftools-tcmalloc-osx-fix.patch patch is no longer needed since
it's automatically "included" into the gperftools source code.

[1] https://github.com/gperftools/gperftools/releases/tag/gperftools-2.10.80
[2] https://github.com/gperftools/gperftools/releases

Change-Id: Ib4916bf64ea90bb148e0abf31779106a57061a15
Reviewed-on: http://gerrit.cloudera.org:8080/20545
Tested-by: Kudu Jenkins
Reviewed-by: Marton Greber <[email protected]>
Reviewed-by: Abhishek Chennaka <[email protected]>
---
M thirdparty/build-definitions.sh
M thirdparty/download-thirdparty.sh
M 
thirdparty/patches/gperftools-Replace-namespace-base-with-namespace-tcmalloc.patch
D thirdparty/patches/gperftools-tcmalloc-osx-fix.patch
M thirdparty/vars.sh
5 files changed, 130 insertions(+), 1,004 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Marton Greber: Looks good to me, but someone else must approve
  Abhishek Chennaka: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/20545
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: Ib4916bf64ea90bb148e0abf31779106a57061a15
Gerrit-Change-Number: 20545
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Yingchun Lai <[email protected]>
Gerrit-Reviewer: Zoltan Martonka <[email protected]>

Reply via email to