Adar Dembo has posted comments on this change. Change subject: parse_metrics_log: merge metrics across entities of same type ......................................................................
Patch Set 1: (4 comments) First time looking at this so mostly a scan, but since it's a standalone script that probably doesn't matter so much. http://gerrit.cloudera.org:8080/#/c/8062/1/src/kudu/scripts/parse_metrics_log.py File src/kudu/scripts/parse_metrics_log.py: PS1, Line 99: if k == 'name': : continue : if k == 'values' or k == 'counts' or k == 'min' or k == 'max' or k == 'mean': : continue Nit: combine? Line 103: else: Nit: no need for the else. Line 161: cur.get('counts', [])) Nit: indentation here. Line 237: if prev_data and ts < prev_data['ts'] + 30: Worth a comment explaining this check. -- To view, visit http://gerrit.cloudera.org:8080/8062 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I442cddd4c5352dcfae30fa23888082d916069d5f Gerrit-PatchSet: 1 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-HasComments: Yes
