Joe McDonnell has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24497 )
Change subject: IMPALA-15010: Patch gperftools to use size_t for thread cache sizes ...................................................................... IMPALA-15010: Patch gperftools to use size_t for thread cache sizes When we allowed a larger total thread cache size in IMPALA-13479, it introduced the possibility of wrapping the int32_t thread cache size. This switches the thread cache sizes to use size_t to avoid any possibility of wrapping. Google TCMalloc had already made this change for their thread caching implementation. This also introduces the latest gperftools 2.18.1 and drops gperftools 2.15. Change-Id: I39d981b55b5bce9f30a4870b2d0b92fb568f9de4 Reviewed-on: http://gerrit.cloudera.org:8080/24497 Reviewed-by: Joe McDonnell <[email protected]> Tested-by: Joe McDonnell <[email protected]> --- M buildall.sh A source/gperftools/gperftools-2.10-patches/0002-Use-size_t-for-thread-cache-sizes.patch D source/gperftools/gperftools-2.15-patches/0001-Remove-1GB-thread-cache-limit.patch A source/gperftools/gperftools-2.18.1-patches/0001-Remove-1GB-thread-cache-limit.patch A source/gperftools/gperftools-2.18.1-patches/0002-Use-size_t-for-thread-cache-sizes.patch 5 files changed, 77 insertions(+), 13 deletions(-) Approvals: Joe McDonnell: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/24497 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: native-toolchain Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I39d981b55b5bce9f30a4870b2d0b92fb568f9de4 Gerrit-Change-Number: 24497 Gerrit-PatchSet: 4 Gerrit-Owner: Joe McDonnell <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Laszlo Gaal <[email protected]> Gerrit-Reviewer: Michael Smith <[email protected]>
