Hello Dan Burkert, David Ribeiro Alves, Todd Lipcon, Alexey Serbin, Kudu Jenkins,
I'd like you to reexamine a change. Please visit http://gerrit.cloudera.org:8080/6945 to look at the new patch set (#5). Change subject: threadpool: new test for pools with no max_threads ...................................................................... threadpool: new test for pools with no max_threads This test ensures that a pool created with effectively no max_threads works as expected. That is: 1. Tokenless tasks should trigger the creation of a new thread. 2. Token-based tasks can create new threads, but only up to the number of tokens submitted against. I intend to use this "feature" to consolidate some Raft thread pools where a num_cpus upper bound may be undesirable (i.e. where tasks submitted to the pools may result in blocking IO). Change-Id: I37d0bd0a05098bcc1a81ec9f1ac33e74e98781e4 --- M src/kudu/util/threadpool-test.cc M src/kudu/util/threadpool.h 2 files changed, 70 insertions(+), 17 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/45/6945/5 -- To view, visit http://gerrit.cloudera.org:8080/6945 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: I37d0bd0a05098bcc1a81ec9f1ac33e74e98781e4 Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Dan Burkert <danburk...@apache.org> Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>