Issue metadata
Sign in to add a comment
|
Merge perf results from obbs shards instead of concat |
||||||||||||||||||||||||
Issue descriptionSimon found some errors with benchmark_total_duration metric that was duplicated across shards since we currently just concat results: https://cs.chromium.org/chromium/src/tools/perf/process_perf_results.py?q=process_perf_results&sq=package:chromium&g=0&l=297 There are other summary metrics that require total benchmark aggregation. Ethan mentioned an add_histogram call that we could use in catapult. I am slightly worried about that given the latency of the upload already.
,
Jun 22 2018
,
Jun 22 2018
,
Jun 25 2018
After we met we have decided to remove benchmark_duration from telemetry. Ethan is going to file a bug to re-investigate the right solution to this. The thought it that we all a histogram per story duration and aggregate in add_reserve_diagnostics, but that is on hold for now. I will file a bug for removing benchmark_duration
,
Jun 25 2018
|
|||||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||||
Comment 1 by eyaich@chromium.org
, Jun 22 2018Status: Duplicate (was: Untriaged)