Hello Adar Dembo, Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

    http://gerrit.cloudera.org:8080/3008

to look at the new patch set (#2).

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
---
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(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/08/3008/2
-- 
To view, visit http://gerrit.cloudera.org:8080/3008
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ie601e914c22b4ca86dc25f4be0c6ec5ef0e94404
Gerrit-PatchSet: 2
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]>

Reply via email to