Closed Bug 549632 Opened 14 years ago Closed 14 years ago

Remove storage cache, which is mostly un/incorrectly used

Categories

(Firefox :: Sync, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: Mardak, Assigned: Mardak)

Details

Attachments

(1 file)

The cache was originally for decreasing the churn of the double loop of creating a local record for the cross product of incoming guids and outgoing guids when reconciling an incoming item. We now actively find a dupe instead of searching for one in the outgoing queue.
Attached patch v1Splinter Review
Assignee: nobody → edilee
Status: NEW → ASSIGNED
Attachment #429744 - Flags: review?
Attachment #429744 - Flags: review? → review?(mconnor)
Comment on attachment 429744 [details] [diff] [review]
v1

I love the smell of cruft removal in the morning, it smells like... victory.
Attachment #429744 - Flags: review?(mconnor) → review+
Remove incorrectly used cache from some engines and clean up references from SyncEngine.
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
Target Milestone: --- → 1.2
Component: Firefox Sync: Backend → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: