Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/23263 )
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]> (cherry picked from commit 9e67e48d64e17e3021d0af2f9dd8c5968dd7d022) Reviewed-on: http://gerrit.cloudera.org:8080/23263 --- M src/kudu/util/thread.cc M src/kudu/util/thread.h 2 files changed, 37 insertions(+), 48 deletions(-) Approvals: Abhishek Chennaka: Looks good to me, approved Alexey Serbin: Verified -- To view, visit http://gerrit.cloudera.org:8080/23263 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.18.x Gerrit-MessageType: merged Gerrit-Change-Id: Id3df2b617a5f648d375e5fb6c6b08e8454e72784 Gerrit-Change-Number: 23263 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
