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)

enhancement

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.

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.

Assignee: nobody → myk
Status: NEW → ASSIGNED
Depends on: 1522638
No longer depends on: 1499238

The bug assignee is inactive on Bugzilla, so the assignee is being reset.

Assignee: myk → nobody
Status: ASSIGNED → NEW

Sorry, there was a problem with the detection of inactive users. I'm reverting the change.

Assignee: nobody → myk
Status: NEW → ASSIGNED
Severity: normal → S3

The bug assignee is inactive on Bugzilla, so the assignee is being reset.

Assignee: myk → nobody
Status: ASSIGNED → NEW

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.