Tim Armstrong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/8114 )
Change subject: Remove unused MemPool::peak_allocated_bytes_ ...................................................................... Remove unused MemPool::peak_allocated_bytes_ The value is not used for anything but there is code devoted to updating it and testing the value. Testing: Ran mem-pool-test to confirm it still works. Change-Id: I99eba01869914c1d1e0a6ed0cab039d904fecc02 Reviewed-on: http://gerrit.cloudera.org:8080/8114 Reviewed-by: anujphadke <[email protected]> Reviewed-by: Dan Hecht <[email protected]> Tested-by: Tim Armstrong <[email protected]> --- M be/src/runtime/mem-pool-test.cc M be/src/runtime/mem-pool.cc M be/src/runtime/mem-pool.h 3 files changed, 2 insertions(+), 32 deletions(-) Approvals: anujphadke: Looks good to me, but someone else must approve Dan Hecht: Looks good to me, approved Tim Armstrong: Verified -- To view, visit http://gerrit.cloudera.org:8080/8114 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I99eba01869914c1d1e0a6ed0cab039d904fecc02 Gerrit-Change-Number: 8114 Gerrit-PatchSet: 2 Gerrit-Owner: Tim Armstrong <[email protected]> Gerrit-Reviewer: Dan Hecht <[email protected]> Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Tim Armstrong <[email protected]> Gerrit-Reviewer: anujphadke <[email protected]>
