Closed
Bug 1185586
Opened 9 years ago
Closed 8 years ago
Investigate ping sending/scheduling on Android
Categories
(Toolkit :: Telemetry, defect, P4)
Tracking
()
RESOLVED
DUPLICATE
of bug 1220171
Tracking | Status | |
---|---|---|
firefox42 | --- | affected |
People
(Reporter: Dexter, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [measurement:client])
We should take some time to investigate the impact of TelemetrySend ping sending scheduler on mobiles, as the scheduler goals are different compared to desktop: on mobile the general goal is to turn on the radio exactly once, ideally when it's needed for other connections. Things to consider: - Minimize bouncing between wakelocks/radio power mode changes - Radio layer packet size (e.g. LTE favours big chunks of data)
Reporter | ||
Updated•9 years ago
|
Comment 1•9 years ago
|
||
We need to check on what is actually expected for ping sending on Android, i don't exactly remember now. Is Telemetry expected to send itself or is a Java service supposed to do this?
Points: --- → 1
Flags: needinfo?(rnewman)
Priority: -- → P3
Updated•9 years ago
|
Whiteboard: [measurement:client]
Comment 2•9 years ago
|
||
All telemetry uploading on Android is currently performed by Gecko. Some parts of the Java frontend *write* ping files, but they don't do the upload themselves. Ideally we'll write a Java uploader: that way we can rely on Android network state events, alarm coalescing, etc. etc., and get one more background service out of Gecko where it doesn't belong.
Flags: needinfo?(rnewman)
Comment 3•9 years ago
|
||
Thanks, then this probably needs no action now and can wait until the mobile Telemetry design discussions happened. We have to address this before fully going for Unified Telemetry on Fennec, but this is not happening yet anyway.
Points: 1 → ---
Priority: P3 → P4
Comment 4•8 years ago
|
||
This is work happening on the mobile Team now, closing this over bug 1220171 et al.
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → DUPLICATE
You need to log in
before you can comment on or make changes to this bug.
Description
•