Todd Lipcon has submitted this change and it was merged. Change subject: Add some more latency-related trace counters ......................................................................
Add some more latency-related trace counters In looking at various RPCs in the new /rpcz profiling mechanism, I noted that a few sources of latency were not well accounted for. This patch adds latency counters for the time spent starting new threads (eg if a threadpool spawns a new worker) and for time spent fsyncing data on disk. A handy new utility class is used to make this kind of counter easy to add. Change-Id: Ie601e914c22b4ca86dc25f4be0c6ec5ef0e94404 Reviewed-on: http://gerrit.cloudera.org:8080/3008 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/util/env_posix.cc M src/kudu/util/thread.cc M src/kudu/util/trace-test.cc M src/kudu/util/trace.h 4 files changed, 49 insertions(+), 1 deletion(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/3008 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie601e914c22b4ca86dc25f4be0c6ec5ef0e94404 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
