Closed Bug 597586 Opened 14 years ago Closed 7 years ago

Increase tab record limit

Categories

(Firefox :: Sync, defect)

defect
Not set
normal

Tracking

()

RESOLVED WORKSFORME

People

(Reporter: mconnor, Unassigned)

References

Details

(Whiteboard: [sync:tabs][sync:scale])

the API allows up to 256k.  The client caps at 20k + encryption overhead because of the SJC-era 32k limit on the server side.  We should figure out how many tabs we want to sync and up the limit by some number.

Getting stats from toby on what % of users actually have records that are > 20k
OS: Windows Vista → All
Priority: -- → P3
Hardware: x86 → All
Target Milestone: 1.6 → ---
bug 600991 comment 0 says this only would affect 0.4% of users, so it's not likely to cause a major increase in memory overhead.

http://mxr.mozilla.org/mozilla-central/source/services/sync/modules/engines/tabs.js#197 should just increase to 100000
Severity: normal → enhancement
Status: NEW → RESOLVED
Closed: 13 years ago
Priority: P3 → P5
Resolution: --- → FIXED
Summary: look at increasing tab record limit → increase tab record limit
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
mconnor: why was this reoppened?
(In reply to Gregory Szorc [:gps] from comment #3)
> mconnor: why was this reoppened?

Looks like it was accidentally closed. The limit is still 20K.
Blocks: 745408
Status: REOPENED → NEW
Summary: increase tab record limit → Increase tab record limit
Whiteboard: [sync:tabs]
Severity: enhancement → normal
Priority: P5 → --
Whiteboard: [sync:tabs] → [sync:tabs][sync:scale]
Status: NEW → RESOLVED
Closed: 13 years ago7 years ago
Resolution: --- → WORKSFORME
Component: Firefox Sync: Backend → Sync
Product: Cloud Services → Firefox
You need to log in before you can comment on or make changes to this bug.