|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
59 bytes, text/x-review-board-request
|Details | Review|
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.
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+
Pushed by firstname.lastname@example.org: https://hg.mozilla.org/integration/autoland/rev/852d4ba2bbb2 Caution against recording Telemetry in hot paths. r=gfritzsche
Status: ASSIGNED → RESOLVED
Last Resolved: a year ago
status-firefox55: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.