Closed Bug 1420196 Opened 7 years ago Closed 1 year ago

Consider changing bookmarks.update to apply transactions to multiple bookmarks to aid performance

Categories

(Toolkit :: Places, enhancement, P3)

Product:
Toolkit
Component:
Places
Type:
enhancement

Tracking

()

Status:
RESOLVED WONTFIX
Tracking Flags:
Tracking Status
firefox59 --- affected

People

(Reporter: standard8, Unassigned)

References

Details

(Keywords: perf, Whiteboard: [fxsearch]) 

In bug 1404909 we're changing bookmarks.update to handle multiple items to update at the same time.

One improvement that we might want to experiment with is to change the executeTransaction wrapper, so that it wraps multiple items, rather than each individual item. See bug 1404909 comment 8.

To do this, we might need to handle chunking as well.

Also, it might be worth at the same time, considering if there is a way to simplify the setAncestorsLastModified calls, as per bug 1404909 comment 9.
Priority: -- → P3
Whiteboard: [fxsearch]
Severity: normal → S3

Not entirely sure this issue is still valid so closing out and can reopen if we observe issues

Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → WONTFIX
You need to log in before you can comment on or make changes to this bug.