Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23260 )
Change subject: [util] explicit relationship between Thread and ThreadMgr ...................................................................... [util] explicit relationship between Thread and ThreadMgr This changelist makes the dependency between Thread and ThreadMgr instances explicit. It makes the code more robust, so it now works even for edge cases of a program's start and shutdown. This patch is a building block for a few more follow-up patches that address the issue of kudu CLI tool crashing at shutdown sometimes. Change-Id: Id3df2b617a5f648d375e5fb6c6b08e8454e72784 Reviewed-on: http://gerrit.cloudera.org:8080/23260 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> --- M src/kudu/util/thread.cc M src/kudu/util/thread.h 2 files changed, 37 insertions(+), 48 deletions(-) Approvals: Alexey Serbin: Verified Abhishek Chennaka: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/23260 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Id3df2b617a5f648d375e5fb6c6b08e8454e72784 Gerrit-Change-Number: 23260 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Zoltan Martonka <[email protected]>
