Marketplace/Performance optimization: Difference between revisions

updated
(add current status)
(updated)
Line 11: Line 11:
# Then we start loading the app images, and there are a lot of them.
# Then we start loading the app images, and there are a lot of them.
All that was on a cable connection, on a desktop. It's going to be worse on mobile.
All that was on a cable connection, on a desktop. It's going to be worse on mobile.
=== Pending ===
* {{Bugzilla|956377}} to enable NewRelic/ElasticSearch deep debugging
* {{Bugzilla|marketplace-perf}} Overall perf tracker for marketplace
* Removing Persona; replacing with FxA


== Want to contribute? ==
== Want to contribute? ==
Line 38: Line 32:
# Filtering apps for "low resource" device
# Filtering apps for "low resource" device
# UI updates to accommodate RTL & other local language requirements (pending target markets)
# UI updates to accommodate RTL & other local language requirements (pending target markets)
Note: We now have the ability to do * [https://rpm.newrelic.com/accounts/315282/dashboard/5293009 NewRelic/ElasticSearch deep debugging]


== Open issues ==
== Open issues ==
<b>These are immediate issues:</b>
<b>These are immediate issues:</b>
* [https://github.com/mozilla/persona/issues/4105 There is an open github issue with Persona about ETags... closed on 2014/10/31]
* [https://github.com/mozilla/persona/issues/4105 There is an open github issue with Persona about ETags... closed (but not resolved) on 2014/10/31]
* [https://github.com/mozilla/persona/issues/3778 ...and one about it failing with packaged apps... closed on 2014/10/31]


<b>These are future issues:</b>
<b>These are future issues:</b>
* Related, [https://bugzilla.mozilla.org/show_bug.cgi?id=983502 having the feature detection API (983502)] as [https://bugzilla.mozilla.org/show_bug.cgi?id=900241 originally described] -- this isn't a Marketplace bug, so we're implementing something else for 1.3 and hope to have this for 1.4, 1.5 at the latest.
* Related, [https://bugzilla.mozilla.org/show_bug.cgi?id=983502 having the feature detection API (983502)] as [https://bugzilla.mozilla.org/show_bug.cgi?id=900241 originally described] -- this isn't a Marketplace bug, so we're implementing something else for 1.3 and hope to have this for 1.4, 1.5 at the latest.


== Open performance bugs ==
== Bugs ==
Tracking bug: {{Bugzilla|marketplace-perf}}.
<bugzilla>
{ "blocks": ["marketplace-perf"], "status": ["UNCONFIRMED", "ASSIGNED", "NEW", "REOPENED"] }
</bugzilla>
 
These bugs have been identified as delivering good performance increases; they are grouped here for ease of consideration.
These bugs have been identified as delivering good performance increases; they are grouped here for ease of consideration.


Line 53: Line 54:


=== overall speed ===
=== overall speed ===
* {{Bugzilla|991301}} (fixed)
* {{Bugzilla|871683}} (wontfix, Server Ops) Note: thus far, SPDY tests have not demonstrated substantial improvements, in part due to system configuration aspects outside of our control. However, these tests were run on the current Marketplace app, which is basically an iframe wrapper. SPDY tests may be more straightforward on the Tarako Marketplace, but as that app is fully packaged it may be irrelevant.
* {{Bugzilla|871683}} (wontfix, Server Ops) Note: thus far, SPDY tests have not demonstrated substantial improvements, in part due to system configuration aspects outside of our control. However, these tests were run on the current Marketplace app, which is basically an iframe wrapper. SPDY tests may be more straightforward on the Tarako Marketplace, but as that app is fully packaged it may be irrelevant.


Line 67: Line 67:
* {{Bugzilla|985293}}
* {{Bugzilla|985293}}
* {{Bugzilla|985804}}
* {{Bugzilla|985804}}
* {{Bugzilla|1087411}} (fixed, next push)
* {{Bugzilla|1087399}} (PR submitted)
* {{Bugzilla|1071468}} (fixed, next push)
* {{Bugzilla|1088361}} (fixed, next push)
* {{Bugzilla|1088362}} (fixed, next push)


=== optimize asset delivery ===
=== optimize asset delivery ===
* {{Bugzilla|874999}} (wontfix)
* {{Bugzilla|874999}} (wontfix)
* {{Bugzilla|987403}} (fixed)
* {{Bugzilla|987851}} (wontfix)
* {{Bugzilla|987851}} (wontfix)
* {{Bugzilla|988674}}
* {{Bugzilla|988674}}
Line 85: Line 79:
* {{Bugzilla|897156}} -- ... becoming a real packaged app.
* {{Bugzilla|897156}} -- ... becoming a real packaged app.


== Performance bugs (general) ==
== General performance bugs (potentially obsolete) ==
This is a listing of Marketplace bugs only, by priority. If a priority has not been applied, you will have to find it directly. [https://bugzilla.mozilla.org/buglist.cgi?priority=--&keywords=perf&keywords_type=allwords&list_id=9800214&resolution=---&query_format=advanced&product=Marketplace Here's a good place to start]
This is a listing of Marketplace bugs only, by priority. If a priority has not been applied, you will have to find it directly. [https://bugzilla.mozilla.org/buglist.cgi?priority=--&keywords=perf&keywords_type=allwords&list_id=9800214&resolution=---&query_format=advanced&product=Marketplace Here's a good place to start]
=== Open P1-P3 ===
<bugzilla>
{
    "product": "Marketplace",
    "status": ["NEW", "ASSIGNED"],
    "resolution": "---",
    "priority": ["P1", "P2", "P3"],
    "keywords": "perf",
    "include_fields": "id, product, component, priority, severity, status, summary, assigned_to, last_change_time"
}
</bugzilla>
=== Open P4-P5 ===
<bugzilla>
{
    "product": "Marketplace",
    "status": ["NEW", "ASSIGNED"],
    "resolution": "---",
    "priority": ["P4", "P5"],
    "keywords": "perf",
    "include_fields": "id, product, component, priority, severity, status, summary, assigned_to, last_change_time"
}
</bugzilla>
=== Resolved: Fixed ===
<bugzilla>
{
    "product": "Marketplace",
    "status": "RESOLVED",
    "resolution": "FIXED",
    "keywords": "perf",
    "include_fields": "id, product, component, priority, severity, status, summary, assigned_to, last_change_time"
}
</bugzilla>
=== Resolved: Deferred ===
<bugzilla>
{
    "product": "Marketplace",
    "status": "RESOLVED",
    "resolution": "WONTFIX",
    "keywords": "perf",
    "include_fields": "id, product, component, priority, severity, status, summary, assigned_to, last_change_time"
}
</bugzilla>




Confirmed users
745

edits