Open
Bug 1544052
Opened 6 years ago
Updated 2 months ago
Consider using `kvstore.sys.mjs` in `PersistentCache.sys.mjs`
Categories
(Firefox :: New Tab Page, enhancement, P3)
Firefox
New Tab Page
Tracking
()
NEW
People
(Reporter: k88hudson, Unassigned)
References
(Depends on 1 open bug)
Details
We're seeing that PersistentCache.jsm is potentially slowing us down at start-up, can we do some preliminary to swap RKV in and see if it would improve the read time at start up?
Reporter | ||
Updated•6 years ago
|
Type: defect → enhancement
Reporter | ||
Updated•6 years ago
|
Priority: -- → P1
Comment 1•6 years ago
|
||
Yes, that's definitely the perfect use case for rkv (kvstore in the JS world).
I think we can make the switch once the rkv memory regression gets fixed (bug 1543795).
Depends on: 1543795
Comment 2•6 years ago
|
||
Looks like we're doing both reads and writes during startup. Is there a way to defer writes for rkv or is it on the caller to request a write later?
Blocks: 1534775
Updated•6 years ago
|
Priority: P1 → P2
Updated•6 years ago
|
Blocks: pocket-newtab-69
Reporter | ||
Updated•6 years ago
|
Blocks: pocket-newtab
Reporter | ||
Updated•6 years ago
|
No longer blocks: pocket-newtab-69
Reporter | ||
Updated•6 years ago
|
No longer blocks: pocket-newtab-68
Updated•5 years ago
|
Assignee: nobody → sdowne
Priority: P2 → P1
Updated•5 years ago
|
Assignee: sdowne → nobody
Priority: P1 → P2
Assignee | ||
Updated•5 years ago
|
Component: Activity Streams: Newtab → New Tab Page
Updated•5 years ago
|
Priority: P2 → P1
Updated•5 years ago
|
Blocks: perf-tweaks
Updated•5 years ago
|
Priority: P1 → P3
Updated•5 years ago
|
No longer blocks: pocket-newtab
Updated•2 years ago
|
Severity: normal → S3
Comment 3•2 months ago
|
||
The SQLite backend for key-value storage that we're adding in bug 1909407 should make this possible, if this is still interesting—but I wouldn't recommend adopting kvstore
before then! 😊
Summary: Consider using RKV for PersistentCache.jsm → Consider using `kvstore.sys.mjs` in `PersistentCache.sys.mjs`
You need to log in
before you can comment on or make changes to this bug.
Description
•