Closed
Bug 995259
Opened 11 years ago
Closed 11 years ago
[tarako] verify point at which number of apps exceeds local storage limit
Categories
(Marketplace Graveyard :: General, defect, P2)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ddurst, Assigned: kngo)
References
Details
(Keywords: perf)
According to Fabrice, the safe ceiling for local storage is 15mb. We should test to determine at what point the cached current data meets/exceeds that limit.
Assignee | ||
Comment 1•11 years ago
|
||
After db-transformer:
4096 apps gives me 11MB preloaded.json
5500 apps gives me 14.7MB preloaded.json
If I'm not misled, dumping into localStorage will yield approx. the same size.
Assignee: nobody → kngo
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 2•11 years ago
|
||
You need to take into account darjeeling own size and most importantly, icons and the first thumbnail for each app.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Reporter | ||
Updated•11 years ago
|
Priority: -- → P2
Assignee | ||
Comment 3•11 years ago
|
||
In theory:
Darjeeling assets (appcache): 210KB
App icon (appcache): ~15KB/app
App preview (appcache): ~40-80KB/app (towards ceiling of range of preview sizes)
App JSON (localStorage): ~3KB/app
(58-98 * apps) + 210 = 15360 (in KB)
Which is about 155 to 261 apps, with previews being the deciding factor.
In practice:
mathieu reported 1.5MB appcache used for 30 apps, which roughly works out to ~300 apps.
Conclusion: we are talking about on the order of a few hundreds of apps.
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → FIXED
Comment 4•11 years ago
|
||
previews and icons should be much lower with the new size for thumbnails and lossless compression (which is enabled now, but we haven't re-generated everything yet)
For instance, for the youtube app:
old thumbnail is 64K, icon is 5.2K
new thumbnail is 19K, icon is 4.9K
I'd expect the range to be 5-10K for icons, 15-25K for previews (obviously some of them can be much bigger depending on complexity)
Assignee | ||
Comment 5•11 years ago
|
||
Did we run it for m.f.c? I saw some 15-20K icons and 60-80K previews? I was basing it off of that.
Comment 6•11 years ago
|
||
It's enabled for new thumbnails and icons on dev, but just for new uploads.
There is a script to re-generate all thubmnails and crush everything but I haven't ran it anywhere but locally yet, which is why you saw bigger icons/thumbnails.
Assignee | ||
Comment 7•11 years ago
|
||
Okay. Given 10K icons, 25K previews, we could have 400 apps.
You need to log in
before you can comment on or make changes to this bug.
Description
•