Closed Bug 623969 Opened 13 years ago Closed 9 months ago

updatePlaces should only update frecency once per URI

Categories

(Toolkit :: Places, defect, P3)

defect

Tracking

()

RESOLVED FIXED
116 Branch

People

(Reporter: sdwilsh, Unassigned)

References

Details

(Keywords: perf, Whiteboard: [snt-scrubbed][places-performance])

Right now History::UpdatePlaces will update frecency after every visit.  If we have duplicate URIs (likely), this is a lot of extra disk I/O that we don't need to do.

Best bet is probably to a hashtable of ids->VisitData (or spec maybe and change the method) and enumerate them.
Depends on: 606966
Keywords: perf
Priority: -- → P3
Severity: normal → S3

This requires to investigate a best approach to the problem.

Whiteboard: [snt-scrubbed][places-performance]
Depends on: 1826946

This should have been addressed by Bug 1826946. Calculation for any batch insert is now delayed.

Status: NEW → RESOLVED
Closed: 9 months ago
Resolution: --- → FIXED
Target Milestone: --- → 116 Branch
You need to log in before you can comment on or make changes to this bug.