Todd Lipcon has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15899 )

Change subject: KUDU-3096: Fix core dump when thread race between 
GetThreadStack and SuperviseThread
......................................................................


Patch Set 1:

One further note I forgot to mention above: the code path here is purposefully 
as short as possible since the spawning thread is blocked waiting for the 
spawned thread to set the tid. In many cases, the spawning thread may be 
latency-sensitive, and is trying to defer work to a thread pool in order to 
return as quickly as possible. So, the more work we do before releasing the tid 
back to the spawner, the worse the latency will be.


--
To view, visit http://gerrit.cloudera.org:8080/15899
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Icc722cd5e8ed4ed668d279f6ec831e4eeb69f955
Gerrit-Change-Number: 15899
Gerrit-PatchSet: 1
Gerrit-Owner: RuiChen <chenrui.m...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-Comment-Date: Mon, 11 May 2020 17:56:14 +0000
Gerrit-HasComments: No

Reply via email to