Closed
Bug 881652
Opened 11 years ago
Closed 7 years ago
[Telemetry] Port JS code to a worker
Categories
(Toolkit :: Telemetry, defect)
Toolkit
Telemetry
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: Yoric, Unassigned)
References
(Blocks 2 open bugs)
Details
(Whiteboard: [Async:team])
Now that chrome workers have a module system (bug 872421), we can start working on implementing libraries.
The objective if this bug is twofold:
- make Telemetry usable from a chrome worker, wrapping this as a module;
- make as much as possible of existing Telemetry clients actually place calls to a Telemetry worker.
Cc-ing Nathan to confirm that it makes sense and to determine which parts of Telemetry can be ported to a worker.
Reporter | ||
Updated•11 years ago
|
Flags: needinfo?(nfroyd)
Comment 1•11 years ago
|
||
Makes sense to me. The I/O is likely the first place to start porting operations to workers. I'm not entirely sure how much other stuff will port easily given dependences on XPCOM.
Flags: needinfo?(nfroyd)
Reporter | ||
Updated•11 years ago
|
Whiteboard: [Async]
Reporter | ||
Comment 2•11 years ago
|
||
So, TelemetryPing.js has a fair share of synchronous I/O, with a flag to ensure that this I/O is specifically synchronous. Nathan, what would I break if I just made all this I/O asynchronous? (except the test that tests that it's synchronous)
Comment 3•11 years ago
|
||
(In reply to David Rajchenbach Teller [:Yoric] from comment #2)
> So, TelemetryPing.js has a fair share of synchronous I/O, with a flag to
> ensure that this I/O is specifically synchronous. Nathan, what would I break
> if I just made all this I/O asynchronous? (except the test that tests that
> it's synchronous)
Should be hardly anything besides the tests.
Reporter | ||
Updated•11 years ago
|
Blocks: WorkForTheWorkers
Updated•11 years ago
|
Whiteboard: [Async] → [Async:team]
Comment 4•7 years ago
|
||
Closing this. We might re-visit it in the future from a fresh Angle. Is worth noting that the Quantum Flow initiative didn't highlight this being a problem though.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•