[kudu-CR] rpcz: break out trace metrics into separate PB fields

["Todd Lipcon (Code Review)" <ge...@cloudera.org>]

Hello Jean-Daniel Cryans,

I'd like you to do a code review.  Please visit

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

to review the following change.

Change subject: rpcz: break out trace metrics into separate PB fields
......................................................................

rpcz: break out trace metrics into separate PB fields

This removes the trace metrics from the textual trace dump and instead adds
some structured metrics information to the /rpcz page. This should be easier
to read in the /rpcz output, and also easier to parse from any future
analysis tools we might build.

I updated the test and also manually verified the new output on a
CreateTableRequestPB on a local instance. Here's the metrics from that call:

 "metrics": [
     {
         "child_path": "txn",
         "key": "apply.queue_time_us",
         "value": 168
     },
     {
         "child_path": "txn",
         "key": "apply.run_wall_time_us",
         "value": 584
     },
     {
         "child_path": "txn",
         "key": "apply.run_cpu_time_us",
         "value": 584
     },
     {
         "child_path": "txn",
         "key": "prepare.run_cpu_time_us",
         "value": 684
     },
     {
         "child_path": "txn",
         "key": "prepare.queue_time_us",
         "value": 168
     },
     {
         "child_path": "txn",
         "key": "prepare.run_wall_time_us",
         "value": 683
     },
     {
         "child_path": "txn",
         "key": "threads_started",
         "value": 2
     },
     {
         "child_path": "txn",
         "key": "thread_start_us",
         "value": 201
     },
     {
         "child_path": "txn",
         "key": "num_ops",
         "value": 2
     },
     {
         "child_path": "txn",
         "key": "replication_time_us",
         "value": 579
     }

Change-Id: I8bd1a249b2f09f8eb63bc64de55147b3d7738a9c
---
M src/kudu/rpc/rpc-test-base.h
M src/kudu/rpc/rpc_introspection.proto
M src/kudu/rpc/rpc_stub-test.cc
M src/kudu/rpc/rpcz_store.cc
M src/kudu/util/trace.cc
M src/kudu/util/trace.h
6 files changed, 77 insertions(+), 3 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/92/3092/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3092
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I8bd1a249b2f09f8eb63bc64de55147b3d7738a9c
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <to...@apache.org>
Gerrit-Reviewer: Jean-Daniel Cryans