Hello Adar Dembo,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/8062
to review the following change.
Change subject: parse_metrics_log: merge metrics across entities of same type
......................................................................
parse_metrics_log: merge metrics across entities of same type
This changes the tool so that all of the tablet metrics get summed
together before reporting. Previously we just reported the info from an
arbitrary tablet (probably the last one to be listed in the log).
This also adds a short circuit such that we skip parsing and merging of
metrics which have not been slated for output. This sped up processing
a metrics file ~10x when reporting only one or two metrics from it.
Lastly, this adds reporting of the max value from histograms.
Change-Id: I442cddd4c5352dcfae30fa23888082d916069d5f
---
M src/kudu/scripts/parse_metrics_log.py
1 file changed, 80 insertions(+), 17 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/62/8062/1
--
To view, visit http://gerrit.cloudera.org:8080/8062
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: I442cddd4c5352dcfae30fa23888082d916069d5f
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
