Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10789 )
Change subject: IMPALA-6425: reduce MemPool max chunk size ...................................................................... IMPALA-6425: reduce MemPool max chunk size IMPALA-2742 capped chunk size doubling to 1MB with the idea that 1MB would be a good value to amortise malloc() overhead. As it turns out, 1MB allocations are significantly more expensive than smaller allocations because they don't have a central freelist in TCMalloc, so the previous choice was misinformed. Note that this doesn't prevent larger allocations from being served from the MemPool, it just caps the automatic doubling behaviour. Perf: Ran single-node TPC-H and targeted perf with scale factor 60. Overall improvement in mean/geomean of 0-2% and no regressions outside of the usual variance. Change-Id: I58cc3031ce592892cb9efe9ab41f07d86468b08c Reviewed-on: http://gerrit.cloudera.org:8080/10789 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M be/src/runtime/mem-pool.h 1 file changed, 3 insertions(+), 2 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/10789 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I58cc3031ce592892cb9efe9ab41f07d86468b08c Gerrit-Change-Number: 10789 Gerrit-PatchSet: 3 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Michael Ho <[email protected]> Gerrit-Reviewer: Tim Armstrong <[email protected]>
