Adar Dembo has posted comments on this change.

Change subject: Bump up min negotiation threads in disk_reservation-itest
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/4575/1//COMMIT_MSG
Commit Message:

PS1, Line 10: pre-spawn those threads in this test.
Why only this test? TSAN causes thread startup to be slow across the board, so 
why not prespawn #if THREAD_SANITIZER universally? Either in the Messenger 
implementation, or by adding the appropriate flags to all mini cluster users.

I still have a preference for bumping FLAGS_rpc_negotiation_timeout_ms, but 
this is orthogonal.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I34c35f0260d5688661972226ff207dc07bd5ecf6
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <mpe...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-HasComments: Yes

Reply via email to