Open
Bug 1254402
Opened 8 years ago
Updated 2 years ago
Use info/configuration endpoint to decide on maximum upload sizes
Categories
(Firefox for iOS :: Sync, defect, P3)
Tracking
()
NEW
Tracking | Status | |
---|---|---|
fxios | 10.0 | --- |
People
(Reporter: rnewman, Unassigned)
References
Details
(Whiteboard: [MobileCore])
See Bug 1250747, Bug 1250189. We should do the same on Android and desktop, too.
Comment 1•8 years ago
|
||
Looking through the discussion https://bugzilla.mozilla.org/show_bug.cgi?id=1250189, would it make sense to implement this prior to implementing atmoic uploads/batching so we can upload batches more intelligently?
Flags: needinfo?(rnewman)
Reporter | ||
Comment 2•8 years ago
|
||
There's no way to use a custom server config on iOS, and these numbers aren't likely to change much, so if we're careful about not shipping batch uploads until the batch API is supported in prod, we can hard-code numbers. The _right_ thing to do is to use this API, and it shouldn't be hard to do… but it doesn't block.
Flags: needinfo?(rnewman)
Updated•8 years ago
|
Priority: -- → P1
Updated•8 years ago
|
Whiteboard: [MobileAS Backlog]
Updated•8 years ago
|
Priority: P1 → --
Updated•8 years ago
|
Whiteboard: [MobileAS Backlog] → [MobileAS]
Updated•8 years ago
|
Priority: -- → P3
Updated•8 years ago
|
Rank: 5
Reporter | ||
Comment 3•7 years ago
|
||
This is now actionable -- the server code is deployed -- so we should do this. Clearing triage flags for re-triage. This allows the client and server to negotiate how many records we put in each HTTP POST (typically more than the default of 100). This means we can do dramatically better wrt power and network usage, allow syncs to complete more quickly, and also let us adapt to network types if we so choose.
Updated•7 years ago
|
Priority: -- → P2
Updated•7 years ago
|
Priority: P2 → P3
Whiteboard: [MobileAS] → [MobileCore]
Updated•7 years ago
|
Updated•7 years ago
|
Updated•7 years ago
|
Rank: 3
Updated•7 years ago
|
Updated•7 years ago
|
Updated•2 years ago
|
Severity: normal → S3
You need to log in
before you can comment on or make changes to this bug.
Description
•