1890582 - Collect telemetry for time spent on purging bounce tracker storage

Status: RESOLVED FIXED

(Core :: Privacy: Anti-Tracking, task, P2)

Description

[Paul Zühlcke [:pbz]]

We should ensure that we don't spent too much time on purging trackers to not clog up the main thread too much. Cleaners (in ClearDataService) usually work async and do disk operations off the main thread - still we might end up with some jank if we're not careful. This is most likely why the old tracker purging mechanism batches deletion of tracker state.

We can use a timing distribution to measure the duration: https://mozilla.github.io/glean/book/reference/metrics/timing_distribution.html

Comment 1

[Manuel Bucher [:manuel]]

Attachment: Bug 1890582 - Collect telemetry for time spent on purging bounce tracker storage r=pbz

Comment 2

[Manuel Bucher [:manuel]]

Attachment: data-request.md

Comment 3

[Manuel Bucher [:manuel]]

Comment on attachment 9397409 [details]
data-request.md

I'd like to change the scope from "Early beta and earl[ier]" to "All populations" including Release. Even though the telemetry probe might get removed before bounce tracking protection gets enabled in release channel (and therefore no collection on release). I think due to this being completely technical data, that shouldn't make a difference to the data-review process. Do I need to resubmit data-request.md?

Comment 4

[Sergio [:sergio]]

Comment on attachment 9397409 [details]
data-request.md

Hi, Manuel, approving to also consider your recent comment about enabling this to all populations.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a197ffa36591
Collect telemetry for time spent on purging bounce tracker storage r=pbz,anti-tracking-reviewers

Comment 6

[Cristian Tuns]

https://hg.mozilla.org/mozilla-central/rev/a197ffa36591