Closed
Bug 1590635
Opened 6 years ago
Closed 1 month ago
LSNG: Add telemetry probes for tracking various performance characteristics
Categories
(Core :: Storage: localStorage & sessionStorage, task, P2)
Core
Storage: localStorage & sessionStorage
Tracking
()
RESOLVED
FIXED
People
(Reporter: janv, Assigned: hsingh)
References
(Blocks 2 open bugs)
Details
Attachments
(2 files)
We need to better understand how our new implementation behaves in the wild. There are some ideas mentioned in bug 1286798 comment 8, but we might need to add more. This is also needed to prevent performance regressions.
|
||
Updated•3 years ago
|
Severity: normal → S3
|
Assignee | |
Updated•1 year ago
|
Assignee: nobody → hsingh
|
|
Assignee | |
Updated•1 year ago
|
Assignee: hsingh → nobody
Keywords: leave-open
|
|
Assignee | |
Updated•1 year ago
|
Assignee: nobody → hsingh
|
|
Assignee | |
Comment 1•1 year ago
|
||
|
||
Updated•1 year ago
|
Attachment #9416890 -
Attachment description: Bug 1590635: Adding a telemetry to report LS request cancellation.r=janv → Bug 1590635: Adding a telemetry to report LS request cancellation.r=janv,#dom-storage-reviewers
|
Reporter | |
Comment 2•1 year ago
|
||
It seems there's currently no good way to report larger debugging information in telemetry except setting a crash annotation and calling MOZ_CRASH or creating a custom ping. A custom ping managed by quota manager looks like a nice, robust and safe way to report debugging information. It would be good to explore that option.
|
|
||
Updated•1 year ago
|
Attachment #9416890 -
Attachment description: Bug 1590635: Adding a telemetry to report LS request cancellation.r=janv,#dom-storage-reviewers → Bug 1590635: Adding a telemetry to report LS first-prepare and any LS request cancellation.r=janv,#dom-storage-reviewers
|
|
||
Updated•1 year ago
|
Attachment #9416890 -
Attachment description: Bug 1590635: Adding a telemetry to report LS first-prepare and any LS request cancellation.r=janv,#dom-storage-reviewers → Bug 1590635: Adding a telemetry to report LS first-prepare latency. r=janv,#dom-storage-reviewers
|
|
Assignee | |
Comment 3•1 year ago
|
||
|
|
||
Updated•1 year ago
|
Attachment #9416890 -
Attachment description: Bug 1590635: Adding a telemetry to report LS first-prepare latency. r=janv,#dom-storage-reviewers → WIP: Bug 1590635: Adding a telemetry to report LS first-prepare latency. r=janv,#dom-storage-reviewers
|
|
||
Updated•1 year ago
|
Attachment #9419308 -
Attachment description: Bug 1590635: Adding a telemetry to report LS request cancellation. r=janv,#dom-storage-reviewers → WIP: Bug 1590635: Adding a telemetry to report LS request cancellation. r=janv,#dom-storage-reviewers
|
|
||
Updated•1 year ago
|
Attachment #9416890 -
Attachment description: WIP: Bug 1590635: Adding a telemetry to report LS first-prepare latency. r=janv,#dom-storage-reviewers → Bug 1590635: Adding a telemetry to report LS first-prepare latency. r=janv,#dom-storage-reviewers
|
|
||
Updated•1 year ago
|
Attachment #9419308 -
Attachment description: WIP: Bug 1590635: Adding a telemetry to report LS request cancellation. r=janv,#dom-storage-reviewers → Bug 1590635: Adding a telemetry to report LS request cancellation. r=janv,#dom-storage-reviewers
|
|
||
Updated•1 year ago
|
Attachment #9416890 -
Attachment description: Bug 1590635: Adding a telemetry to report LS first-prepare latency. r=janv,#dom-storage-reviewers → Bug 1590635: Adding a telemetry to report LS perparedatastoreop processing and latency. r=janv,#dom-storage-reviewers
|
|
||
Updated•1 year ago
|
Attachment #9416890 -
Attachment description: Bug 1590635: Adding a telemetry to report LS perparedatastoreop processing and latency. r=janv,#dom-storage-reviewers → Bug 1590635: Adding a telemetry to report LS database and perparedatastore processing time. r=janv,#dom-storage-reviewers
Pushed by hsingh@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/4d3eeb7fe9d6
Adding a telemetry to report LS database and perparedatastore processing time. r=janv,dom-storage-reviewers
https://hg.mozilla.org/integration/autoland/rev/bebc2a827b41
Adding a telemetry to report LS request cancellation. r=janv,dom-storage-reviewers
|
||
Comment 5•1 year ago
|
||
| bugherder | ||
|
||
Comment 6•1 month ago
|
||
The leave-open keyword is there and there is no activity for 6 months.
:hsingh, maybe it's time to close this bug?
For more information, please visit BugBot documentation.
Flags: needinfo?(hsingh)
|
|
Assignee | |
Comment 7•1 month ago
|
||
I have already added few new LS metrics as part of this ticket. And I don't have anymore to add at this point.
I think it's make sense to close this task and later create new one on the need-basis.
Flags: needinfo?(hsingh)
|
|
Assignee | |
Updated•1 month ago
|
Status: NEW → RESOLVED
Closed: 1 month ago
Resolution: --- → FIXED
|
||
Updated•29 days ago
|
Keywords: leave-open
You need to log in
before you can comment on or make changes to this bug.
Description
•