Closed Bug 1428510 Opened 7 years ago Closed 4 years ago

'modified' and 'created' fields should be NOT NULL on the History table

Categories

(Firefox for Android Graveyard :: Data Providers, enhancement, P5)

enhancement

Tracking

(Not tracked)

RESOLVED INCOMPLETE

People

(Reporter: Grisha, Unassigned)

References

Details

Stemming out of Bug 1428165, which is specifically concerned with the history table. Other tables will benefit from this as well, but it's easier to attack this one table at a time - at the expense of a schema migration churn. 'modified' timestamp is currently used by Sync for change tracking, and so this work will involve a migration that's concerned with its impact on history Sync (we don't want to trigger an upload of records unnecessarily). Unfortunately, due to SQLite's limitations, this migration will involve going through a temp table and copying a whole lot of history records - all at the upgrade time. This needs some careful thought, and will benefit greatly from having a limited set of history records (correctly working expiration for a release or so is a pre-req, so post-Bug 1428165).
I believe the "this requires copying everything" issue is why we haven't done this in the past. For users with limited space, this will permanently lock them out of Firefox: we won't have space to create the new table, and we can't continue without upgrading the database. You could write a conditional migration that's OK with failure, but then what's the point?
Re-triaging per https://bugzilla.mozilla.org/show_bug.cgi?id=1473195 Needinfo :susheel if you think this bug should be re-triaged.
Priority: -- → P5
We have completed our launch of our new Firefox on Android. The development of the new versions use GitHub for issue tracking. If the bug report still reproduces in a current version of [Firefox on Android nightly](https://play.google.com/store/apps/details?id=org.mozilla.fenix) an issue can be reported at the [Fenix GitHub project](https://github.com/mozilla-mobile/fenix/). If you want to discuss your report please use [Mozilla's chat](https://wiki.mozilla.org/Matrix#Connect_to_Matrix) server https://chat.mozilla.org and join the [#fenix](https://chat.mozilla.org/#/room/#fenix:mozilla.org) channel.
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → INCOMPLETE
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in before you can comment on or make changes to this bug.