Skip to content
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

Per VHOST mod_pagespeed_statistics #430

Closed
GoogleCodeExporter opened this issue Apr 6, 2015 · 4 comments
Closed

Per VHOST mod_pagespeed_statistics #430

GoogleCodeExporter opened this issue Apr 6, 2015 · 4 comments

Comments

@GoogleCodeExporter
Copy link

@GoogleCodeExporter GoogleCodeExporter commented Apr 6, 2015

It would be helpful if the 'mod_pagespeed_statistics' handler could be enhanced 
to provide, per VHOST granularity as opposed to global statistics.

If there is an alternative, please let me know.

Thanks

Original issue reported on code.google.com by hayes...@gmail.com on 7 May 2012 at 5:27

@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Apr 6, 2015

I think this is possible.  We debated about making statistics work this way 
when initially designing them, but didn't for two reasons:
  - the initial implementation of statistics was not scalable to have a
    distinct set for each vhost (but this is no longer the case)
  - we rationalized that to monitor server health you wanted to see the statistics
    aggregated together.  However it might be reasonable to bind them to the
    cache directory.
The main challenge now is figuring when we could get this done.

Original comment by jmara...@google.com on 8 May 2012 at 2:29

@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Apr 6, 2015

Original comment by jmara...@google.com on 31 May 2012 at 7:35

  • Changed state: Accepted
@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Apr 6, 2015

Code in place. Need to add docs, too, though.

Original comment by morlov...@google.com on 10 Sep 2012 at 6:00

  • Changed state: Started
  • Added labels: Milestone-v23, release-note
@GoogleCodeExporter
Copy link
Author

@GoogleCodeExporter GoogleCodeExporter commented Apr 6, 2015

Docs staged (will be pushed once a version with the code is part of a release)

Original comment by morlov...@google.com on 5 Nov 2012 at 4:38

  • Changed state: Fixed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
1 participant
You can’t perform that action at this time.