Michael Smith has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/20796 )
Change subject: Revert "IMPALA-11805: Fix LLVM memory manager bytes allocated" ...................................................................... Revert "IMPALA-11805: Fix LLVM memory manager bytes allocated" This reverts commit 8874ea07b68c09cee3c4e300b70706f3c5ea40af. Reason for revert: other work on IMPALA-11805 will stop using the memory manager's size for the codegen cache, and changing this had broader implications on query memory tracking that need to be accounted for. It started causing memory reservation failures in tests. Change-Id: I7f7a6b21b3a9fc7c9e675c8d3349725c4863f744 Reviewed-on: http://gerrit.cloudera.org:8080/20796 Reviewed-by: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Tested-by: Michael Smith <michael.sm...@cloudera.com> --- M be/src/codegen/llvm-codegen-cache-test.cc M be/src/codegen/llvm-codegen-test.cc M be/src/codegen/mcjit-mem-mgr.h M be/src/thirdparty/llvm/SectionMemoryManager.cpp M be/src/thirdparty/llvm/SectionMemoryManager.h 5 files changed, 22 insertions(+), 40 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved Michael Smith: Verified -- To view, visit http://gerrit.cloudera.org:8080/20796 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I7f7a6b21b3a9fc7c9e675c8d3349725c4863f744 Gerrit-Change-Number: 20796 Gerrit-PatchSet: 3 Gerrit-Owner: Michael Smith <michael.sm...@cloudera.com> Gerrit-Reviewer: Andrew Sherman <asher...@cloudera.com> Gerrit-Reviewer: Csaba Ringhofer <csringho...@cloudera.com> Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com> Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com> Gerrit-Reviewer: Yida Wu <wydbaggio...@gmail.com>