Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22195 )
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]> (cherry picked from commit 33fd5e482b4854d23247af903c4263afc9931e59) Reviewed-on: http://gerrit.cloudera.org:8080/22195 --- 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/22195 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.18.x Gerrit-MessageType: merged Gerrit-Change-Id: I8f6a019c7c033c6c3dfef59b0f037e78f8264e68 Gerrit-Change-Number: 22195 Gerrit-PatchSet: 2 Gerrit-Owner: Mahesh Reddy <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
