Closed
Bug 1065923
Opened 10 years ago
Closed 1 years ago
Simplify TelemetryVFS by removing telemetry gathering and rename it BaseVFS
Categories
(Toolkit :: Storage, defect, P3)
Tracking
()
RESOLVED
FIXED
115 Branch
Tracking | Status | |
---|---|---|
firefox115 | --- | fixed |
People
(Reporter: mak, Assigned: mak)
References
Details
(Whiteboard: DWS_NEXT)
Attachments
(2 files)
Most of the telemetry is reporting useless or not measurable stuff, and thus many probes will just expire in version 40.
We should simplify this code and eventually evaluate to rename it to quotaVFS or storageVFS since telemetry won't be anymore the primary purpose of it.
See the various probes:
- MOZ_SQLITE_*
- MOZ_STORAGE_*
Comment 1•10 years ago
|
||
Sounds good to me. I would prefer QuotaVFS.
Assignee | ||
Updated•8 years ago
|
Priority: -- → P3
Updated•7 years ago
|
Assignee: nobody → bugmail
Blocks: 730495
Status: NEW → ASSIGNED
Summary: Simplify TelemetryVFS → Simplify TelemetryVFS by removing telemetry gathering and rename it QuotaVFS
Updated•7 years ago
|
Comment 2•6 years ago
|
||
Andrew: would it be okay if we move this to backlog? DWS_NEXT perhaps?
Flags: needinfo?(bugmail)
Comment 3•6 years ago
|
||
Yeah, I thought I'd already removed myself from this (probably in an open tab somewhere), but this is probably a good medium-term cleanup as far as QuotaManager is concerned (after WSIF).
Assignee: bugmail → nobody
Status: ASSIGNED → NEW
Flags: needinfo?(bugmail)
Whiteboard: DWS_NEXT
Updated•4 years ago
|
Updated•2 years ago
|
Severity: normal → S3
Assignee | ||
Updated•2 years ago
|
Summary: Simplify TelemetryVFS by removing telemetry gathering and rename it QuotaVFS → Simplify TelemetryVFS by removing telemetry gathering and rename it BaseVFS
Assignee | ||
Updated•2 years ago
|
Assignee | ||
Comment 4•2 years ago
|
||
Assignee | ||
Comment 5•2 years ago
|
||
As previously discussed, we're not using this telemetry at the moment, we used
it in the past to study main thread IO.
Simplifying the VFS allows us to more easily experiment with other settings
in the future.
Depends on D172015
Assignee | ||
Comment 6•2 years ago
|
||
This is not going to land before the merge.
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/a32ebe969fa8
Rename Storage TelemetryVFS to BaseVFS. r=janv
https://hg.mozilla.org/integration/autoland/rev/656d0aa7e02c
Remove histograms from Storage BaseVFS. r=janv,mossop,dom-storage-reviewers
Comment 8•1 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a32ebe969fa8
https://hg.mozilla.org/mozilla-central/rev/656d0aa7e02c
Status: ASSIGNED → RESOLVED
Closed: 1 years ago
status-firefox115:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 115 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•