Closed Bug 1569516 Opened 4 months ago Closed 3 months ago

Remove very old calendar storage migration code

Categories

(Calendar :: Provider: Local Storage, task)

task
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: darktrojan, Assigned: darktrojan)

Details

Attachments

(1 file)

It's highly unlikely that any user would still be using a storage.sdb database, and it's unreasonable to expect that we could still migrate from it to the current database.

Removing the migration code also prevents us from synchronously opening the file at start-up. We never check whether the file exists, so even if it doesn't we create it and then try to read it.

Attachment #9081179 - Flags: review?(philipp)
Comment on attachment 9081179 [details] [diff] [review]
1569516-remove-old-schema-1.diff

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

\o/ to removing old code!
Attachment #9081179 - Flags: review?(philipp) → review+
Keywords: checkin-needed

Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/580987860caa
Remove very old calendar storage migration code. r=philipp

Status: ASSIGNED → RESOLVED
Closed: 3 months ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → 70
You need to log in before you can comment on or make changes to this bug.