Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/21414 )
Change subject: [common] switch from unique_lock to lock_guard ...................................................................... [common] switch from unique_lock to lock_guard To simplify working with STL synchronization primitives, this patch changes from std::unique_lock to std::lock_guard where appropriate. In addition, IWYU and ClangTidy's feedback has been addressed. Change-Id: I72d86ca730113ec0652154d5ce509fc2e479befb Reviewed-on: http://gerrit.cloudera.org:8080/21414 Tested-by: Alexey Serbin <ale...@apache.org> Reviewed-by: Zoltan Chovan <zcho...@cloudera.com> Reviewed-by: Abhishek Chennaka <achenn...@cloudera.com> --- M src/kudu/clock/hybrid_clock.cc M src/kudu/consensus/consensus_peers.cc M src/kudu/consensus/consensus_queue.cc M src/kudu/consensus/log_cache.cc M src/kudu/integration-tests/tombstoned_voting-stress-test.cc M src/kudu/rpc/rpcz_store.cc M src/kudu/security/tls_context.cc M src/kudu/server/webserver.cc M src/kudu/tablet/lock_manager-test.cc M src/kudu/tablet/lock_manager.cc M src/kudu/tablet/tablet.cc M src/kudu/tserver/scanners.cc M src/kudu/util/maintenance_manager.cc M src/kudu/util/trace_metrics.h 14 files changed, 36 insertions(+), 31 deletions(-) Approvals: Alexey Serbin: Verified Zoltan Chovan: Looks good to me, but someone else must approve Abhishek Chennaka: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/21414 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I72d86ca730113ec0652154d5ce509fc2e479befb Gerrit-Change-Number: 21414 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <ale...@apache.org> Gerrit-Reviewer: Attila Bukor <abu...@apache.org> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Chovan <zcho...@cloudera.com>