Mike Percy has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9253 )
Change subject: KUDU-2291 (part 2): Add a /stacks page ...................................................................... KUDU-2291 (part 2): Add a /stacks page This adds a simple /stacks web page which dumps the currently running threads in a plain text format. Since we often have a lot of idle threadpool threads sitting around in the "wait for work" state, the output collapses all threads with matching stacks and only displays the stack once, making it more suitable for human consumption. Example output from a local kudu-master: https://gist.github.com/b64739ee5fb146ea1953380f57b996c4 Longer term we may want to integrate stack-tracing capability into the /threadz view as well, but for now I left this as a low-level utility which doesn't access the thread manager, etc. I left a few TODOs for further enhancements, but I've already found this helpful for understanding some perf anomalies while playing with YCSB, so let's get it committed and improve as we go. Change-Id: I8b8f6d50d44e40fd51357fdbfd8f9ba2ebaa724b Reviewed-on: http://gerrit.cloudera.org:8080/9253 Tested-by: Todd Lipcon <t...@apache.org> Reviewed-by: Mike Percy <mpe...@apache.org> --- M src/kudu/integration-tests/linked_list-test-util.h M src/kudu/server/default_path_handlers.cc M src/kudu/util/debug-util.cc 3 files changed, 106 insertions(+), 9 deletions(-) Approvals: Todd Lipcon: Verified Mike Percy: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/9253 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I8b8f6d50d44e40fd51357fdbfd8f9ba2ebaa724b Gerrit-Change-Number: 9253 Gerrit-PatchSet: 6 Gerrit-Owner: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>
