1673663 - Implement C++ and JS APIs for the Timing Distribution Metric Type

Status: RESOLVED FIXED

(Toolkit :: Telemetry, task, P1)

Description

[Chris H-C :chutten]

Following the docs, implement the C++ and JS parts for the Timing Distribution metric type.

Pay close attention to how you manage the TimerId. Use types where you can.

Comment 1

[Chris H-C :chutten]

Attachment: Bug 1673663 - Move FOG Timing Distribution to RLB impl r?janerik!

Comment 2

[Chris H-C :chutten]

Attachment: Bug 1673663 - JS and C++ API for FOG Timing Distribution r?janerik!

Depends on D101639

Comment 3

[Chris H-C :chutten]

Oh bother, a TSan lock inversion: https://treeherder.mozilla.org/logviewer?job_id=326625964&repo=try&lineNumber=3015-3188

We already dealt with this for events and Timespans, looks like we get to do it for Timing Distributions now.

Thank goodness for TSan.

Comment 4

[Chris H-C :chutten]

Attachment: Bug 1673663 - Update Glean to 33.10.2 to get lock order fix r?TravisLong!

Depends on D101640

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/8739882b0152
Move FOG Timing Distribution to RLB impl r=janerik
https://hg.mozilla.org/integration/autoland/rev/afd854af3414
JS and C++ API for FOG Timing Distribution r=janerik
https://hg.mozilla.org/integration/autoland/rev/fbce1827bb9e
Update Glean to 33.10.2 to get lock order fix r=TravisLong

Comment 6

[Cristian Brindusan [:cbrindusan]]

https://hg.mozilla.org/mozilla-central/rev/8739882b0152
https://hg.mozilla.org/mozilla-central/rev/afd854af3414
https://hg.mozilla.org/mozilla-central/rev/fbce1827bb9e