Todd Lipcon has submitted this change and it was merged. 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 Reviewed-on: http://gerrit.cloudera.org:8080/8062 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/scripts/parse_metrics_log.py 1 file changed, 91 insertions(+), 20 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/8062 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I442cddd4c5352dcfae30fa23888082d916069d5f 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]>
