Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21856 )
Change subject: [tablet] reduce lock contention in OpTracker::Release() ...................................................................... [tablet] reduce lock contention in OpTracker::Release() Since MemTracker::Release() runs tcmalloc's GC-ing which might block for some time under heavy concurrent activity, it makes sense to call it outside of the critical section that guards the map with pending operations for the tablet. Also, avoid double lookups in that map, re-using the iterator for erasing the element. Change-Id: Ic5e4298ccabe2f70500765d63083cc817b7a5f95 Reviewed-on: http://gerrit.cloudera.org:8080/21856 Reviewed-by: Abhishek Chennaka <achenn...@cloudera.com> Tested-by: Alexey Serbin <ale...@apache.org> --- M src/kudu/tablet/ops/op_tracker.cc 1 file changed, 13 insertions(+), 7 deletions(-) Approvals: Abhishek Chennaka: Looks good to me, approved Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/21856 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ic5e4298ccabe2f70500765d63083cc817b7a5f95 Gerrit-Change-Number: 21856 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Yifan Zhang <chinazhangyi...@163.com>
