Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10320 )
Change subject: IMPALA-6507: remove --disable_mem_pools debug feature ...................................................................... IMPALA-6507: remove --disable_mem_pools debug feature Save some maintenance overhead by simplifying memory allocation code paths. ASAN poisoning provides the same general functionality and is on by default. Change-Id: I78062569448fed0d076ec506eb8e097ce44d9fea Reviewed-on: http://gerrit.cloudera.org:8080/10294 Reviewed-by: Dan Hecht <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> Reviewed-on: http://gerrit.cloudera.org:8080/10320 Reviewed-by: Sailesh Mukil <[email protected]> Reviewed-by: Tim Armstrong <[email protected]> --- M be/src/common/global-flags.cc M be/src/runtime/bufferpool/buffer-allocator.cc M be/src/runtime/free-pool.h M be/src/runtime/io/disk-io-mgr.cc M be/src/runtime/mem-pool.cc 5 files changed, 4 insertions(+), 40 deletions(-) Approvals: Impala Public Jenkins: Verified Sailesh Mukil: Looks good to me, but someone else must approve Tim Armstrong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/10320 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: 2.x Gerrit-MessageType: merged Gerrit-Change-Id: I78062569448fed0d076ec506eb8e097ce44d9fea Gerrit-Change-Number: 10320 Gerrit-PatchSet: 2 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Sailesh Mukil <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
