Tim Armstrong has posted comments on this change.

Change subject: PREVIEW: IMPALA-3104: Catch exception on failure to create 
thread
......................................................................


Patch Set 2:

I think we should probably figure out the general direction and priority of 
this. It seems like this is the direction we want to go.

It would be great to have this fixed but it seems like there are a lot of 
changes to implement this globally, so it may be quite time consuming.

Another question is whether we should consider using pthreads directly instead 
of boost's thread. I believe boost is taking the error code from pthreads and 
converting it into an exception.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I93629bbec08a0ea362794cd739503476e0caa1c5
Gerrit-PatchSet: 2
Gerrit-Project: Impala
Gerrit-Branch: cdh5-trunk
Gerrit-Owner: Sailesh Mukil <[email protected]>
Gerrit-Reviewer: Michael Ho <[email protected]>
Gerrit-Reviewer: Tim Armstrong <[email protected]>
Gerrit-HasComments: No

Reply via email to