1433807 - Removed unused `mirror.locations` index

Status: RESOLVED FIXED

(Firefox :: Sync, enhancement, P1)

Description

[Lina Butler (ex-Mozilla)]

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 1

[Lina Butler (ex-Mozilla)]

Attachment: Bug 1433807 - Removed unused `mirror.locations` index.

Since the synced bookmark mirror is currently disabled by default, we
can do this without a migration.

Review commit: https://reviewboard.mozilla.org/r/216366/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/216366/

Comment 2

[Thom Chiovoloni [:tcsc] (ex-moco)]

Comment on attachment 8946413 [details]
Bug 1433807 - Removed unused `mirror.locations` index.

https://reviewboard.mozilla.org/r/216366/#review222170

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/481e5fcc2b5e
Removed unused `mirror.locations` index. r=tcsc

Comment 4

[Raul Gurzau (:RaulG)]

https://hg.mozilla.org/mozilla-central/rev/481e5fcc2b5e