Hello David Ribeiro Alves, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/10693
to look at the new patch set (#2).
Change subject: parse_metrics_log: update to the new format
......................................................................
parse_metrics_log: update to the new format
The new diagnostics logs report more than just metrics, and thus, output
a bit differently than they did in simpler times. This patch updates the
parsing to be compatible with the new logs.
The changes this patch addresses:
- each metrics log line had a different number of fields
- the metrics log will only emit metrics that have changed; as such,
the processing of metrics has been changed; where we previously filled
in NaNs for missing data, we now pull from the previous snapshot, or
fill in 0 if one doesn't exist
- the histogram-parsing code seemed a bit off; this patch updates it
Change-Id: If11b7ecc93a3f64db3b7c1994f47308b3ec44029
---
M src/kudu/scripts/parse_metrics_log.py
1 file changed, 64 insertions(+), 20 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/93/10693/2
--
To view, visit http://gerrit.cloudera.org:8080/10693
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If11b7ecc93a3f64db3b7c1994f47308b3ec44029
Gerrit-Change-Number: 10693
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
