1321396 - Allow skipped records by default for sync engines

Status: RESOLVED FIXED

(Firefox :: Sync, defect, P1)

Description

[Thom Chiovoloni [:tcsc] (ex-moco)]

If a record is too large to be uploaded to the server, currently most engines fail to sync instead of skipping that record. This is overly conservative, and instead we should only fail in cases where a partial sync is worse than no sync at all (like bookmarks, clients, prefs -- since it's a single record, etc.)

Patch incoming.

Comment 1

[Thom Chiovoloni [:tcsc] (ex-moco)]

Attachment: Bug 1321396 - Skip oversized records without aborting the sync by default in sync engines.

Review commit: https://reviewboard.mozilla.org/r/96678/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/96678/

Comment 2

[Mark Hammond (PTO; back Oct-3) [:markh] [:mhammond]]

Comment on attachment 8815881 [details]
Bug 1321396 - Skip oversized records without aborting the sync by default in sync engines.

https://reviewboard.mozilla.org/r/96678/#review96910

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/16a80745ac45
Skip oversized records without aborting the sync by default in sync engines. r=markh

Comment 4

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/16a80745ac45

Comment 5

[Mark Hammond (PTO; back Oct-3) [:markh] [:mhammond]]

Comment on attachment 8815881 [details]
Bug 1321396 - Skip oversized records without aborting the sync by default in sync engines.

Approval Request Comment
[Feature/Bug causing the regression]: bug 1253051
[User impact if declined]: Users with massive records to sync will fail to sync at all
[Is this code covered by automated tests?]: yes
[Has the fix been verified in Nightly?]: yes, and Aurora
[Needs manual test from QE? If yes, steps to reproduce]: No 
[List of other uplifts needed for the feature/fix]: None
[Is the change risky?]: Low risk.
[Why is the change risky/not risky?]: limited to sync, has been on central for some time
[String changes made/needed]: None

Sorry for the delay here - this slipped off our radar and the request should have been made earlier.

Comment 6

[Julien Cristau [:jcristau]]

Comment on attachment 8815881 [details]
Bug 1321396 - Skip oversized records without aborting the sync by default in sync engines.

most sync engines allow skipping big records rather than failing, beta52+, should be in b3

Comment 7

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-beta/rev/b293d6e395a1