1834100 - Consider creating a timing distribution for the heaviest process in SearchSERPTelemetryChild

Status: RESOLVED FIXED

(Firefox :: Search, task, P2)

Description

[James Teow [:jteow]]

Parsing the anchors and categorizing DOM elements can be an expensive operation.

We've been doing testing on a couple machines to do sanity checks that the processing time isn't crazy, but we could be more exhaustive by gathering info using a Glean timing distribution.

Relevant links:

• Timing Distribution
• A patch that does it for a feature

Comment 1

[James Teow [:jteow]]

Attachment: Bug 1834100 - Use timing distribution for categorizing components in SearchSERPTelemetryChild - r?standard8,scunnane

Depends on D178911

Comment 2

[James Teow [:jteow]]

Attachment: data-review-bug-1834100.md

Comment 3

[Jared Hirsch [:jhirsch] (he/him) (Needinfo please)]

Comment on attachment 9336411 [details]
data-review-bug-1834100.md

data-review+, LGTM

---

1. Is there or will there be documentation that describes the schema for the ultimate data set in a public, complete, and accurate way?

Yes, the usual Firefox Glean documentation.

2. Is there a control mechanism that allows the user to turn the data collection on and off?

Yes, the usual Firefox telemetry opt-out.

3. If the request is for permanent data collection, is there someone who will monitor the data over time?

Yes, the search team will monitor via the [e-mail] alias.

4. Using the category system of data types on the Mozilla wiki, what collection type of data do the requested measurements fall under?

Category 1: technical data.

5. Is the data collection request for default-on or default-off?

Default-on.

6. Does the instrumentation include the addition of any new identifiers?

No.

7. Is the data collection covered by the existing Firefox privacy notice?

Yes.

8. Does the data collection use a third-party collection tool?

No.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6e9d51287c53
Use timing distribution for categorizing components in SearchSERPTelemetryChild - r=Standard8

Comment 5

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/6e9d51287c53