New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Track image rewriting time in mod_pagespeed_variables #586

Closed
GoogleCodeExporter opened this Issue Apr 6, 2015 · 3 comments

Comments

Projects
None yet
1 participant
@GoogleCodeExporter
Copy link

GoogleCodeExporter commented Apr 6, 2015

We currently track a bunch of image processing metrics in 
mod_pagespeed_variables, but we don't track actual time spent doing image 
rewriting.  It would be useful to make this available as a cumulative image 
processing time and a histogram of individual image rewrite times.  This would 
inform a bunch of performance tuning, both for better install defaults and for 
individual site owners adjusting their configuration.

Original issue reported on code.google.com by jmaes...@google.com on 10 Dec 2012 at 3:51

@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Original comment by jmaes...@google.com on 10 Dec 2012 at 3:52

@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

Reassigning to Huibao as he's been looking at some image instrumentation 
issues, and I think the metrics now exist so could be histogrammed / maybe are 
being histogrammed now.

Original comment by jmaes...@google.com on 11 Mar 2013 at 1:33

@GoogleCodeExporter

This comment has been minimized.

Copy link

GoogleCodeExporter commented Apr 6, 2015

In the next release, you will see CPU time for image rewriting in a stat: 
image_rewrite_latency_total_ms

Original comment by jmara...@google.com on 31 Jan 2014 at 3:58

  • Changed state: Fixed
  • Added labels: Milestone-v31, release-note
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment