Closed
Bug 1607908
Opened 4 years ago
Closed 4 years ago
Add a pingsender feature to support sending multiple pings at once
Categories
(Toolkit :: Telemetry, enhancement, P1)
Toolkit
Telemetry
Tracking
()
RESOLVED
FIXED
mozilla74
Tracking | Status | |
---|---|---|
firefox74 | --- | fixed |
People
(Reporter: chutten, Assigned: gsvelto)
References
(Blocks 2 open bugs)
Details
Attachments
(1 file)
Invocations of pingsender processes can be slow. Instead of invoking it once for each ping at shutdown, we could put them all into a batch (see bug 1607586) and pass all the paths to a single pingsender process.
If the sending of any one ping fails, pingsender can just bail out at that point with a non-zero exit status. We'll still eventually get everything when Firefox next starts up and checks for pending pings.
Assignee | ||
Comment 1•4 years ago
|
||
I've got a WIP patch that implements the functionality but I'm still writing the tests.
Assignee | ||
Comment 2•4 years ago
|
||
Updated•4 years ago
|
Attachment #9120452 -
Attachment description: Bug 1607908 - Add support for sending multiple pings via a single invocation of the pingsender → Bug 1607908 - Add support for sending multiple pings via a single invocation of the pingsender r=chutten
Pushed by gsvelto@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/eeddcfbedc06 Add support for sending multiple pings via a single invocation of the pingsender r=chutten
Comment 4•4 years ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox74:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla74
You need to log in
before you can comment on or make changes to this bug.
Description
•