Closed
Bug 1515096
Opened 5 years ago
Closed 6 months ago
use transactions to speed up migration of notifications to key-value storage
Categories
(Core :: DOM: Notifications, enhancement, P3)
Core
DOM: Notifications
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: myk, Unassigned)
References
Details
Once transactions are available to kvstore.jsm consumers (bug 1499238), we should use one to speed up migration of notifications data to the key-value storage engine provided by kvstore.jsm. See bug 1515094 for the migration itself; this bug is specifically about using transactions to speed up that migration.
Updated•5 years ago
|
Priority: -- → P3
Reporter | ||
Comment 1•5 years ago
|
||
Although kvstore doesn't yet have transactions, it does now have bulk insert (bug 1522638), and that'd speed up migration of notifications to the same extent, so we should use it. Updating the dependency accordingly, and taking the bug.
Comment 2•2 years ago
|
||
The bug assignee is inactive on Bugzilla, so the assignee is being reset.
Assignee: myk → nobody
Status: ASSIGNED → NEW
Comment 3•2 years ago
|
||
Sorry, there was a problem with the detection of inactive users. I'm reverting the change.
Assignee: nobody → myk
Status: NEW → ASSIGNED
Updated•2 years ago
|
Severity: normal → S3
Comment 4•6 months ago
|
||
The bug assignee is inactive on Bugzilla, so the assignee is being reset.
Assignee: myk → nobody
Status: ASSIGNED → NEW
Comment 5•6 months ago
|
||
Given that bug 1499238 will not happen, we should close this, too. If we have any knowledge about this being a performance problem these days (as we now use rkv in safe-mode) , we should start a new investigation on how to improve this.
Status: NEW → RESOLVED
Closed: 6 months ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•