2012188 - Remove telemetry.event_counts

Status: RESOLVED FIXED

(Toolkit :: Telemetry, task, P1)

Description

[Chris H-C :chutten]

telemetry.event_counts is in the top 25 probes by cost. Its purpose was to supply a cheaper alternative to events. Events would be off, but even when off we'd count how often they'd happen. That would allow for many analyses (volume, frequency) without having to receive events from everyone all of the time. Unfortunately, we never really used this, and Legacy Telemetry events have been always on for years.

As it's not in use and won't be, let's remove it.

This bug is about merely removing the instrumentation, but can also include removing the event summarization feature as well if that's straightforward and/or satisfying.

Comment 1

[Chris H-C :chutten]

Attachment: Bug 2012188 - Remove Legacy Telemetry event summarization r?TravisLong

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/44572f04f41e
https://hg.mozilla.org/integration/autoland/rev/6d8603c7972d
Remove Legacy Telemetry event summarization r=TravisLong,extension-reviewers,omc-reviewers,emcminn,rpl

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/14b4722c7d20
https://hg.mozilla.org/integration/autoland/rev/b57233e3ce36
Revert "Bug 2012188 - Remove Legacy Telemetry event summarization r=TravisLong,extension-reviewers,omc-reviewers,emcminn,rpl" for causing non-unified build bustages on TelemetryScalar

Comment 4

[Cosmin Sabou [:CosminS]]

Backed out for causing non-unified build bustages on TelemetryScalar

Push with failures

Failure log

Backout link

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://github.com/mozilla-firefox/firefox/commit/fb5e88acb601
https://hg.mozilla.org/integration/autoland/rev/5ffe8a1fb623
Remove Legacy Telemetry event summarization r=TravisLong,extension-reviewers,omc-reviewers,emcminn,rpl

Comment 6

[Cristina Horotan [:chorotan]]

https://hg.mozilla.org/mozilla-central/rev/5ffe8a1fb623