Add note on performance to Telemetry collection docs

RESOLVED FIXED in Firefox 55

Status

()

Toolkit
Telemetry
P1
normal
RESOLVED FIXED
a year ago
a year ago

People

(Reporter: gfritzsche, Assigned: chutten)

Tracking

(Blocks: 1 bug)

Trunk
mozilla55
Points:
1

Firefox Tracking Flags

(firefox55 fixed)

Details

(Whiteboard: [measurement:client])

MozReview Requests

()

Submitter Diff Changes Open Issues Last Updated
Loading...
Error loading review requests:

Attachments

(1 attachment)

(Reporter)

Description

a year ago
Our assumption is that Telemetry is not accumulated from busy/hot loops etc.
Instead we expect users to accumulate locally in those cases and e.g. just Accumulate() the result after.

We should document this expectation explicitly, with notes in histogram, scalar & event docs.
(Reporter)

Updated

a year ago
Priority: P2 → P1
(Reporter)

Updated

a year ago
Priority: P1 → P2
(Reporter)

Updated

a year ago
Priority: P2 → P1
(Reporter)

Updated

a year ago
Points: --- → 1
(Assignee)

Updated

a year ago
Assignee: nobody → chutten
Status: NEW → ASSIGNED
Comment hidden (mozreview-request)
(Reporter)

Comment 2

a year ago
mozreview-review
Comment on attachment 8855426 [details]
bug 1339019 - Caution against recording Telemetry in hot paths.

https://reviewboard.mozilla.org/r/127272/#review130390

Thanks, that looks good enough for now, we can revisit this later if needed.
Attachment #8855426 - Flags: review?(gfritzsche) → review+

Comment 3

a year ago
Pushed by chutten@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/852d4ba2bbb2
Caution against recording Telemetry in hot paths. r=gfritzsche
https://hg.mozilla.org/mozilla-central/rev/852d4ba2bbb2
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
status-firefox54: affected → ---
You need to log in before you can comment on or make changes to this bug.