Hello Adar Dembo, Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/6856 to look at the new patch set (#4). Change subject: log: shut down appender thread when idle ...................................................................... log: shut down appender thread when idle This changes the log appender thread to be based on submitting tasks to a pool, rather than a thread which is always running. See the comments on Log::AppendThread in log.cc for notes on the design. This also adds a new test case in mt-log-test which triggers the various complexities of the appender lifecycle. I manually checked coverage using LOG(WARNING) statements to make sure it does exercise the various cases. Change-Id: Id40a4cfcec96198b537c2f50be7ff1204faf96d2 --- M src/kudu/consensus/log-test-base.h M src/kudu/consensus/log-test.cc M src/kudu/consensus/log.cc M src/kudu/consensus/log.h M src/kudu/consensus/mt-log-test.cc M src/kudu/util/blocking_queue-test.cc M src/kudu/util/blocking_queue.h M src/kudu/util/threadpool.h 8 files changed, 301 insertions(+), 102 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/56/6856/4 -- To view, visit http://gerrit.cloudera.org:8080/6856 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Id40a4cfcec96198b537c2f50be7ff1204faf96d2 Gerrit-PatchSet: 4 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-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>