Closed
Bug 1126567
Opened 9 years ago
Closed 9 years ago
[Raptor] Prime application in setup of cold launch phase
Categories
(Firefox OS Graveyard :: Gaia::PerformanceTest, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Eli, Assigned: Eli)
References
Details
(Keywords: perf)
Attachments
(1 file)
The purpose of cold launch testing is to measure the launch performance of an application as a new process. By running the cold launch tests from a fresh flash scenario, we include latency from an app's initial launch that may skew the time to be much longer than normal, whether this be from DB initialization or other IO operations. These measurements should be accounted for in the "first-time" suites, but not the cold launch ones. To achieve this, we should change the setup of the cold launch suite to launch an app and then close it before starting any of the actual test runs. This should eliminate the bulk of extra latency between first-time and cold-launch numbers.
Comment 1•9 years ago
|
||
Excellent - yeah I was thinking we would need to 'throw out' the first value because of this; this is an even better solution.
Assignee | ||
Comment 2•9 years ago
|
||
Attachment #8557219 -
Flags: review?(rwood)
Assignee | ||
Updated•9 years ago
|
Status: NEW → ASSIGNED
FWIW, if you need a simple term for this, I've heard it called "priming" in relationship to website caches. I've been calling our issue the same. But yeah, this is a much better solution than chucking the dirty result.
Assignee | ||
Comment 4•9 years ago
|
||
Me likely. I'll rename the method to `prime` before merging.
Summary: [Raptor] Pre-launch and close application in setup of cold launch phase → [Raptor] Prime application in setup of cold launch phase
Comment 5•9 years ago
|
||
Comment on attachment 8557219 [details] [review] Link to Github pull-request: https://github.com/mozilla-b2g/raptor/pull/9 Awesome. Tested by running the launch test on flame and the b2g emulator. Verified the app is primed/pre-launched successfully, and the data from the pre-launch is not included in the raptor.log and visualization, as expected.
Attachment #8557219 -
Flags: review?(rwood) → review+
Assignee | ||
Comment 6•9 years ago
|
||
Merged. https://github.com/mozilla-b2g/raptor/commit/dbcba0a3f866d5fd06a6089654ecf7a2bcc52af4
Assignee | ||
Updated•9 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•