Remove MAX_SCHEMA_RETRY define now that we have SQLite 3.7.13

RESOLVED FIXED in Firefox 52

Status

()

Toolkit
Storage
P3
normal
RESOLVED FIXED
5 years ago
10 months ago

People

(Reporter: Ben Turner (not reading bugmail, use the needinfo flag!), Assigned: mak)

Tracking

Trunk
mozilla52
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox52 fixed)

Details

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

IndexedDB needed a larger MAX_SCHEMA_RETRY in order to work before SQLite 3.7.13, but now it shouldn't be needed.
Depends on: 789645
I don't think it's worth it.
Status: NEW → RESOLVED
Last Resolved: 10 months ago
Resolution: --- → WONTFIX
(Assignee)

Updated

10 months ago
Duplicate of this bug: 877657
oops, we can indeed remove it cause 3.7.17 set the default to 50 and we use 25.
Status: RESOLVED → REOPENED
Priority: -- → P3
Resolution: WONTFIX → ---
(Assignee)

Updated

10 months ago
Assignee: nobody → mak77
Comment hidden (mozreview-request)

Comment 5

10 months ago
mozreview-review
Comment on attachment 8807879 [details]
Bug 764573 - Remove MAX_SCHEMA_RETRY since now the default value is larger.

https://reviewboard.mozilla.org/r/90866/#review90594
Attachment #8807879 - Flags: review?(bugmail) → review+

Comment 6

10 months ago
Pushed by mak77@bonardo.net:
https://hg.mozilla.org/integration/autoland/rev/d7f727af4b9f
Remove MAX_SCHEMA_RETRY since now the default value is larger. r=asuth

Comment 7

10 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/d7f727af4b9f
Status: REOPENED → RESOLVED
Last Resolved: 10 months ago10 months ago
status-firefox52: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla52
You need to log in before you can comment on or make changes to this bug.