The default bug view has changed. See this FAQ.

Add Telemetry for hit rate & revalidation rate

RESOLVED FIXED in mozilla9

Status

()

Core
Networking: Cache
RESOLVED FIXED
6 years ago
4 years ago

People

(Reporter: nwgh, Assigned: nwgh)

Tracking

Trunk
mozilla9
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 1 obsolete attachment)

We want to collect telemetry for the following bits of information:

Cache hit (without revalidation from the server)
Successful revalidation (hit with network overhead)
Failed revalidation (we have an entry but after asking the server it can no longer be used)
Cache miss (no entry in the cache)
Created attachment 560448 [details] [diff] [review]
patch from work time
Attachment #560448 - Flags: review?(mcmanus)
Comment on attachment 560448 [details] [diff] [review]
patch from work time

Nick, can you just fix up our 80 column violations before checking it in?
Attachment #560448 - Flags: review?(mcmanus) → review+
Created attachment 560612 [details] [diff] [review]
Patch update

Fixes line length. Carry forward r+
Attachment #560448 - Attachment is obsolete: true
Attachment #560612 - Flags: review+
Keywords: checkin-needed
Presuming try goes green, will push to inbound first thing tomorrow :-)
https://tbpl.mozilla.org/?tree=Try&usebuildbot=1&rev=329eee3b938e
Status: NEW → ASSIGNED
Keywords: checkin-needed
Version: unspecified → Trunk
https://hg.mozilla.org/integration/mozilla-inbound/rev/0b31c67e1423
Target Milestone: --- → mozilla9
https://hg.mozilla.org/mozilla-central/rev/0b31c67e1423
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED

Comment 7

4 years ago
Comment from Mozilla:test
You need to log in before you can comment on or make changes to this bug.