1413515 - Remove e10s_cohort from derived datasets, queries, and jobs.

Status: RESOLVED FIXED

(Data Platform and Tools :: General, enhancement, P1)

Description

[Mark Reid [:mreid]]

Per bug 1413296, this field is no longer relevant, and is being removed on the client.

We should remove it elsewhere in the pipeline as well. Copying my comment from the above bug:

This information is in several derived datasets, but as :chutten mentioned, I don't think its removal on the client will break anything (since we're already handling older data that doesn't have it).

We should review and remove it from
- Telemetry Batch View    https://github.com/mozilla/telemetry-batch-view
- python_mozetl           https://github.com/mozilla/python_mozetl/
- firefox-data-docs       https://github.com/mozilla/firefox-data-docs

And notify people that they should update their queries/jobs
- in re:dash              https://sql.telemetry.mozilla.org/queries/48445/source
- on atmo

Comment 1

[Frank Bertsch [:frank]]

also: telemetry-streaming https://github.com/mozilla/telemetry-streaming/

Comment 2

[Daniel Thorn [:relud]]

:chutten should I also be removing e10s_enabled and e10s_multi_processes?

in telemetry-batch-view e10s_enabled is used in determining quantum_ready for MainSummaryView, should I leave it in just for that view, modify quantum_ready to not reference e10s_enabled, or remove quantum_ready?

Comment 3

[Chris H-C :chutten]

e10s_enabled and e10s_multi_processes are still fine to include. They reflect things that users may have changed, and will be useful if we in future find out that some large %ge of the population is operating on non-standard e10s settings.

Comment 4

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/telemetry-batch-view/pull/345

Comment 5

[GitHub Bugzilla PR Linker]

Attachment: Link to GitHub pull-request: https://github.com/mozilla/python_mozetl/pull/172

Comment 6

[Daniel Thorn [:relud]]

more limited set of re:dash candidates for notifying people to update queries: https://sql.telemetry.mozilla.org/queries/49155/source

Comment 7

[Daniel Thorn [:relud]]

resolving this and moving the work of notifying people to its own bug