Issue metadata
Sign in to add a comment
|
3.7% regression in system_health.memory_desktop at 601281:601335 |
||||||||||||||||||||
Issue descriptionSee the link to graphs below.
,
Nov 8
📍 Pinpoint job started. https://pinpoint-dot-chromeperf.appspot.com/job/142059a1e40000
,
Nov 8
📍 Found a significant difference after 1 commit. https://pinpoint-dot-chromeperf.appspot.com/job/142059a1e40000 cc: Rate limit raster metric queries. by khushalsagar@chromium.org https://chromium.googlesource.com/chromium/src/+/e898b99eac0cf5db379cf8c7e216d772095a7448 memory:chrome:all_processes:reported_by_os:system_memory:private_footprint_size: 1.41e+08 → 1.443e+08 (+3.287e+06) Understanding performance regressions: http://g.co/ChromePerformanceRegressions Benchmark documentation link: https://bit.ly/system-health-benchmarks
,
Nov 8
There were a couple of timing changes that came with adding this metric that improved memory in some cases. For instance, I see [1] in the timeline before this change which caused an improvement that was countered with the change in #3. Overall any memory changes from adding this metric was unintended and the rate limiting just brings us back to where we were initially. [1]: https://chromium.googlesource.com/chromium/src/+/da82bcce69bf5384435976a1945c9cb3270c5abb
,
Nov 8
,
Nov 8
Issue 903119 has been merged into this issue. |
|||||||||||||||||||||
►
Sign in to add a comment |
|||||||||||||||||||||
Comment 1 by 42576172...@developer.gserviceaccount.com
, Nov 8