Closed
Bug 1253112
Opened 9 years ago
Closed 8 years ago
[iOS] Upload bookmark records atomically
Categories
(Firefox for iOS :: Sync, defect, P1)
Tracking
()
RESOLVED
FIXED
Iteration:
1.3
Tracking | Status | |
---|---|---|
fxios-v6.0 | --- | fixed |
People
(Reporter: rnewman, Assigned: sleroux, Mentored)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [sync-atomic][MobileAS])
Attachments
(1 file)
This is the iOS equivalent of Bug 1253051. See that bug for details.
Reporter | ||
Updated•9 years ago
|
Reporter | ||
Updated•9 years ago
|
Updated•8 years ago
|
Priority: -- → P1
Whiteboard: [data-integrity]
Assignee | ||
Updated•8 years ago
|
Blocks: activity-stream-ios-fxa
Assignee | ||
Comment 1•8 years ago
|
||
Relevant API documentation: https://github.com/mozilla-services/docs/pull/60/files
Assignee | ||
Comment 2•8 years ago
|
||
I can take a stab at this. Putting :rnewman as a mentor :)
Assignee: nobody → sleroux
Mentor: rnewman
Assignee | ||
Comment 3•8 years ago
|
||
Attachment #8773442 -
Flags: feedback?(rnewman)
Reporter | ||
Updated•8 years ago
|
Attachment #8773442 -
Flags: feedback?(rnewman) → feedback+
Comment 4•8 years ago
|
||
I don't understand swift, but one thing that stands out: The way the spec is written, we can't know the batch semantics of the server until we make the first post (and thus have already enqueued a number of records). It's not clear this is handled here - or maybe the intent was to deduce the server semantics from the result of the info/configuration endpoint and fail if we thought the server was going to give us a batch but didn't? Best I can tell this patch doesn't hit info/configuration at all, nor use the batch limits it returns, so maybe that will become clearer in later versions of the patch?
Updated•8 years ago
|
Summary: Upload bookmark records atomically → [iOS] Upload bookmark records atomically
Reporter | ||
Comment 5•8 years ago
|
||
On iOS we don't support custom servers, so we don't have to worry too much about non-supported batch operations; see Bug 1254402 for using i/conf.
Updated•8 years ago
|
Whiteboard: [data-integrity] → [data-integrity][MobileAS s1.1]
Comment 6•8 years ago
|
||
(In reply to Richard Newman [:rnewman] from comment #5)
> On iOS we don't support custom servers, so we don't have to worry too much
> about non-supported batch operations; see Bug 1254402 for using i/conf.
FWIW, I believe the plan for rolling out the server side of this is "slowly" :) It's tricky to fully simulate a production load, so I believe ops will enable this per-node and disable it if any problems are found (then presumably tweak, rinse and repeat). In practice, this means a patch that assumes batch semantics can only land once the config is enabled for all nodes and ops are confident they will never need to turn it off.
(Richard says something similar in bug 1254402 comment 2, but I thought it worth explicitly calling out here)
Updated•8 years ago
|
Whiteboard: [data-integrity][MobileAS s1.1] → [sync-atomic][MobileAS s1.1]
Updated•8 years ago
|
Whiteboard: [sync-atomic][MobileAS s1.1] → [sync-atomic][MobileAS s1.2]
Updated•8 years ago
|
Status: NEW → ASSIGNED
Updated•8 years ago
|
Whiteboard: [sync-atomic][MobileAS s1.2] → [sync-atomic][MobileAS s1.3]
Updated•8 years ago
|
Priority: P1 → P2
Updated•8 years ago
|
Whiteboard: [sync-atomic][MobileAS s1.3] → [sync-atomic][MobileAS]
Updated•8 years ago
|
Priority: P2 → P1
Assignee | ||
Comment 7•8 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
status-fxios-v6.0:
--- → fixed
Resolution: --- → FIXED
Updated•8 years ago
|
Iteration: --- → 1.3
You need to log in
before you can comment on or make changes to this bug.
Description
•