Closed Bug 1039961 Opened 5 years ago Closed 5 years ago
[B2G][OTA] The migration process is performed twice if the user enters in the tutorial
There is nor dataloss neither issues but we are consuming a lot of resources performing this task twice when it is not needed
Assignee: nobody → crdlc
Status: NEW → ASSIGNED
blocking-b2g: --- → 2.0?
The migrator should be clever and to know when the migration was performed or not independent on number of calls
Comment on attachment 8457831 [details] Github pull request Seems a bit sketchy to depend on this specific value, but I can't think of a better solution, so WFM.
Attachment #8457831 - Flags: review?(kgrandon) → review+
In the end the task is to migrate collections/bookmarks to datastores + save the old-layout. If we have the layout stored, we have been migrated. Datastores without revisiionId does not mean that the migration was not performed because the layout could not have nor collections neither bookmarks. So the only way to know if the migration was performed or not is this. So I don't understand "a bit sketchy" :) (In reply to Kevin Grandon :kgrandon from comment #2) > Comment on attachment 8457831 [details] > Github pull request > > Seems a bit sketchy to depend on this specific value, but I can't think of a > better solution, so WFM.
As discussed online, the user impact is low on that bug. The memory is freed after the tutorial. I don't think we should block on that. This a nice to have though. Do not hesitate to ask for approval after the fix is landed.
blocking-b2g: 2.0? → ---
QA Whiteboard: [VH-FL-blocking-][VH-FC-blocking-]
Comment on attachment 8457831 [details] Github pull request This looks good to me, thanks!
Attachment #8457831 - Flags: review?(carmen.jimenezcabezas) → review+
Comment on attachment 8457831 [details] Github pull request This task is huge because it works with indexedDB and two datastores. If we perform this task twice we will have a memory impact opening the tutorial and also we are consuming more battery when it is not needed. The patch only checks if the migration was done previously and adds an early return. [Approval Request Comment] [Bug caused by] (feature/regressing bug #): no regression [User impact] if declined: no [Testing completed]: yes [Risk to taking this patch] (and alternatives if risky): close to null [String changes made]: no
Attachment #8457831 - Flags: approval-gaia-v2.0?(bbajaj)
Attachment #8457831 - Flags: approval-gaia-v2.0?(bbajaj) → approval-gaia-v2.0+
You need to log in before you can comment on or make changes to this bug.