Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/21276
Change subject: [tablet] one less memory allocation in CBTree::count() ...................................................................... [tablet] one less memory allocation in CBTree::count() While reviewing [1], I noticed there is room for improvement in CBTree. One trivial one is avoiding calls to 'new' when an object might be allocated on the stack. This makes sense since in some intensive Kudu workloads we might see significant lock contention in tcmalloc, and removing needless calls to 'new' helps to relieve that, even if it's not in the hot path. [1] https://gerrit.cloudera.org/#/c/21127/ Change-Id: I01a68e0427b399db92b33c910185654d195150a5 --- M src/kudu/tablet/concurrent_btree.h 1 file changed, 7 insertions(+), 7 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/76/21276/1 -- To view, visit http://gerrit.cloudera.org:8080/21276 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I01a68e0427b399db92b33c910185654d195150a5 Gerrit-Change-Number: 21276 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
