Adar Dembo has posted comments on this change. Change subject: maintenance_manager: schedule work immediately when threads are free ......................................................................
Patch Set 1: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/6815/1/src/kudu/util/maintenance_manager.cc File src/kudu/util/maintenance_manager.cc: PS1, Line 221: or it is time to run another op. The "shutting down" part is fairly intuitive, but this part isn't. It might be useful to expand on this and explain how it relates to the two parts of the condition (comparing running_ops to num_threads and force_sleep_next_iter). -- To view, visit http://gerrit.cloudera.org:8080/6815 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I63c4b48f5f02f3a1d3a8964993e78037ce72b1da Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: Yes