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)

enhancement

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.

I've got a WIP patch that implements the functionality but I'm still writing the tests.

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
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla74
Blocks: 1610757
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: