Closed Bug 1433807 Opened 2 years ago Closed 2 years ago

Removed unused `mirror.locations` index

Categories

(Firefox :: Sync, enhancement, P1)

enhancement

Tracking

()

RESOLVED FIXED
Firefox 60
Tracking Status
firefox60 --- fixed

People

(Reporter: Lina, Assigned: Lina)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

I don't think any of the mirror's queries actually use this index. We do a full table scan to build the complete tree, then stuff the merged tree into `mergeStates`.

Since we haven't turned this on for anyone yet, I think we can just drop the index without a migration.
Comment on attachment 8946413 [details]
Bug 1433807 - Removed unused `mirror.locations` index.

https://reviewboard.mozilla.org/r/216366/#review222170
Attachment #8946413 - Flags: review?(tchiovoloni) → review+
Priority: -- → P1
Pushed by kcambridge@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/481e5fcc2b5e
Removed unused `mirror.locations` index. r=tcsc
https://hg.mozilla.org/mozilla-central/rev/481e5fcc2b5e
Status: NEW → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
Assignee: nobody → lina
You need to log in before you can comment on or make changes to this bug.