1232276 - Repeat sync if fetchLimit was reached

Status: RESOLVED WONTFIX

(Firefox OS Graveyard :: Sync, defect, P1)

Description

[Michiel de Jong [:michielbdejong]]

For history sync, when more than 50 new records are available through Syncto, we only retrieve the 50 oldest ones, and ignore/skip the other records. When sync is run again, only newer records than the skipped ones will be fetched.

Once https://github.com/Kinto/kinto.js/pull/279 lands, we should change this behavior so that no records are ever skipped, it will simply take more sync runs (scheduled hourly or when the user clicks 'Sync Now') to get up to date with the remote data set.

It would also be nice to give the user some feedback saying '50 of 15,000 records retrieved', and maybe even make the 'Sync Now' button repeat automatically until all records have been fetched.

Comment 1

[Michiel de Jong [:michielbdejong]]

I'm preparing a patch based on https://github.com/Kinto/kinto.js/pull/299

Comment 2

[GitHub Autolander Bot]

Attachment: [gaia] michielbdejong:1232276-repeat-sync > mozilla-b2g:master

Comment 3

[GitHub Autolander Bot]

Attachment: [gaia] michielbdejong:1232276-repeat-sync-v2.5 > mozilla-b2g:v2.5

Comment 4

[Michiel de Jong [:michielbdejong]]

Comment on attachment 8704102 [details] [review]
[gaia] michielbdejong:1232276-repeat-sync-v2.5 > mozilla-b2g:v2.5

This patch is now outdated.

After bug 1236863 is merged, we can just take the patch from michielbdejong:1232276-repeat-sync and cherry-pick that to the v2.5 branch.

Comment 5

[Fernando Jiménez Moreno [:ferjm]]

Comment on attachment 8704100 [details] [review]
[gaia] michielbdejong:1232276-repeat-sync > mozilla-b2g:master

Thank you Michiel! The code LGTM. I left a couple of comments on the PR. 

It seems that this patch can't be accepted for 2.5 TV until we fix bug 1237568 and does not make much sense for the phone until bug 1231933 is fixed, right? 

Also, the fact that we have to start over the sync process every time it is canceled plus the fact that the user is unable to see the history or bookmarks because of bug 1237581 is a shame. So I'd like to fix these two issues as well if possible before accepting this patch.

Comment 6

[Michiel de Jong [:michielbdejong]]

Blocked on bug 1231933 for the phone, and on bug 1237568 for the TV, so let's postpone.

Comment 7

[Fernando Jiménez Moreno [:ferjm]]

We discussed about this in our stand up meeting and we decided to 2.5- this issue. Instead of taking this approach we preferred to take bug 1237570 which we consider a safer option. In the end, even if it is a nice to have, not having the whole history synced is not a big deal for the user. We don't have autocomplete (bug 1231933) or history search features and we don't expect an user to scroll down a 2k items list looking for a specific URL, so we consider that syncing the latest 500 history records is good enough for now.

Comment 8

[Sean Lee [:seanlee][:weilonge]]

Based on Comment 7, it can be closed temporarily.