Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22181 )
Change subject: KUDU-613: Remove use of vectors under lock ...................................................................... KUDU-613: Remove use of vectors under lock Currently, the implementation of the SLRU cache returns a vector of temporarily evicted entries to be reinserted later. There are performance concerns with this approach as it allocates and deallocates memory under the lock. This patch refactors the code to not use a vector to deal with the temporarily evicted entries. Change-Id: I8f6a019c7c033c6c3dfef59b0f037e78f8264e68 Reviewed-on: http://gerrit.cloudera.org:8080/22181 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Alexey Serbin <[email protected]> --- M src/kudu/util/slru_cache.cc M src/kudu/util/slru_cache.h 2 files changed, 44 insertions(+), 57 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/22181 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I8f6a019c7c033c6c3dfef59b0f037e78f8264e68 Gerrit-Change-Number: 22181 Gerrit-PatchSet: 7 Gerrit-Owner: Mahesh Reddy <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Kurt Deschler <[email protected]> Gerrit-Reviewer: Mahesh Reddy <[email protected]>
