Issue metadata
Sign in to add a comment
|
139.8%-305% regression in blink_perf.owp_storage at 534517:534678 |
||||||||||||||||||||||
Issue descriptionThis suite goes from 2 minutes to 11 minutes, which is way too much
,
Feb 12 2018
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/14b3085d840000
,
Feb 12 2018
Daniel: can you look into the benchmark runtime regression?
,
Feb 12 2018
These are the new benchmarks being added.
,
Feb 12 2018
Daniel: how many tests did you add? Our device budget is very constraint, so it would be great if you can look into reducing the runtime
,
Feb 12 2018
Ok, I'll reduce the test count from 12 to 4 by removing some of our strategies, and it will definitely bring this time down. When do you need this done by?
,
Feb 12 2018
This is affecting how soon people can get perf results, so the sooner you can address this the better. Fixing this in 2 weeks is reasonable to me.
,
Feb 12 2018
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/14b3085d840000 [BlobStorage] Adding size configurations to perftests by dmurph@chromium.org https://chromium.googlesource.com/chromium/src/+/1131391c410a6d01c584c4449accc7e1b1258b06 Understanding performance regressions: http://g.co/ChromePerformanceRegressions
,
Feb 12 2018
I think this is WAI, given that we changed the tests.
,
Feb 13 2018
#9: please see #5 & #6
,
Feb 13 2018
Sorry I didn't read the whole thread carefully and thank you for fixing my mistake!
,
Feb 21 2018
,
Jan 3
Reduced the tests, just didn't put this bug in the cl number. Now we have only 4, and they are pretty small.
,
Jan 16
(6 days ago)
,
Jan 16
(6 days ago)
|
|||||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Feb 12 2018