Closed Bug 707955 Opened 9 years ago Closed 9 years ago

Tags should avoid results overhead

Categories

(Toolkit :: Places, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla11

People

(Reporter: mak, Assigned: mak)

References

Details

(Keywords: perf)

Attachments

(1 file)

While it's the cleaner way to do that with the current API, results have an overhead we can't currently pay on hot paths doing lots of tags operations.
It's just simpler to use queries, this also allows to reduce the queries traffic.
Attached patch patch v1.0Splinter Review
Use direct queries, and some caching trick
Attachment #579502 - Flags: review?(dietrich)
Comment on attachment 579502 [details] [diff] [review]
patch v1.0

Review of attachment 579502 [details] [diff] [review]:
-----------------------------------------------------------------

r=me.
Attachment #579502 - Flags: review?(dietrich) → review+
https://hg.mozilla.org/mozilla-central/rev/ae823406bcb3
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.