Skip to content

Commit c79d6d5

Browse files
authored
Bug 1697602 Remove derived tables for AET (#1894)
* Bug 1697602 Remove derived tables for AET * Update GRAVEYARD doc
1 parent fd03e1d commit c79d6d5

File tree

20 files changed

+8
-745
lines changed

20 files changed

+8
-745
lines changed

GRAVEYARD.md

+8
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,14 @@
22

33
This document records interesting code that we've deleted for the sake of discoverability for the future.
44

5+
## 2021-03 Account Ecosystem Telemetry (AET) derived tables
6+
7+
- [Removal PR](https://github.com/mozilla/bigquery-etl/pull/1894)
8+
9+
AET was never released except for a short test in the beta population,
10+
and now the project has been decommissioned, so there is no longer
11+
any need for these derived tables.
12+
513
## 2020-04 Fenix baseline_daily and clients_last_seen
614

715
- [Removal PR](https://github.com/mozilla/bigquery-etl/pull/925)

bigquery_etl/shredder/config.py

-31
Original file line numberDiff line numberDiff line change
@@ -72,9 +72,6 @@ def fields(self) -> Tuple[str, ...]:
7272
USER_ID = "user_id"
7373
POCKET_ID = "pocket_id"
7474
SHIELD_ID = "shield_id"
75-
ECOSYSTEM_CLIENT_ID = "ecosystem_client_id"
76-
ECOSYSTEM_CLIENT_ID_HASH = f"{ECOSYSTEM_CLIENT_ID}_hash"
77-
DESKTOP_ECOSYSTEM_CLIENT_ID = f"payload.{ECOSYSTEM_CLIENT_ID}"
7875
PIONEER_ID = "pioneer_id"
7976
ID = "id"
8077
CFR_ID = f"COALESCE({CLIENT_ID}, {IMPRESSION_ID})"
@@ -95,14 +92,6 @@ def fields(self) -> Tuple[str, ...]:
9592
f" UNNEST([{CLIENT_ID}, {IMPRESSION_SRC.field}]) AS `_",
9693
field="_",
9794
)
98-
ECOSYSTEM_CLIENT_ID_HMAC_SRC = DeleteSource(
99-
table="account_ecosystem_restricted.ecosystem_client_id_deletion_v1",
100-
field=ECOSYSTEM_CLIENT_ID_HASH,
101-
)
102-
ECOSYSTEM_CLIENT_ID_SRC = DeleteSource(
103-
table="account_ecosystem_restricted.ecosystem_client_id_deletion_v1",
104-
field=ECOSYSTEM_CLIENT_ID,
105-
)
10695
FXA_HMAC_SRC = DeleteSource(
10796
table="firefox_accounts_derived.fxa_delete_events_v1", field="hmac_user_id"
10897
)
@@ -138,8 +127,6 @@ def fields(self) -> Tuple[str, ...]:
138127
DESKTOP_SRC,
139128
IMPRESSION_SRC,
140129
CFR_SRC,
141-
ECOSYSTEM_CLIENT_ID_HMAC_SRC,
142-
ECOSYSTEM_CLIENT_ID_SRC,
143130
FXA_HMAC_SRC,
144131
FXA_SRC,
145132
]
@@ -274,21 +261,6 @@ def fields(self) -> Tuple[str, ...]:
274261
user_id_target(
275262
table="firefox_accounts_derived.fxa_users_services_last_seen_v1"
276263
): FXA_SRC,
277-
# account ecosystem telemetry (AET)
278-
DeleteTarget(
279-
table="telemetry_stable.account_ecosystem_v4", field=DESKTOP_ECOSYSTEM_CLIENT_ID
280-
): ECOSYSTEM_CLIENT_ID_SRC,
281-
DeleteTarget(
282-
table="firefox_accounts_stable.account_ecosystem_v1", field=ECOSYSTEM_CLIENT_ID
283-
): ECOSYSTEM_CLIENT_ID_SRC,
284-
DeleteTarget(
285-
table="account_ecosystem_derived.ecosystem_client_id_lookup_v1",
286-
field=ECOSYSTEM_CLIENT_ID_HASH,
287-
): ECOSYSTEM_CLIENT_ID_HMAC_SRC,
288-
DeleteTarget(
289-
table="account_ecosystem_derived.desktop_clients_daily_v1",
290-
field=ECOSYSTEM_CLIENT_ID_HASH,
291-
): ECOSYSTEM_CLIENT_ID_HMAC_SRC,
292264
# legacy mobile
293265
DeleteTarget(
294266
table="telemetry_stable.core_v1",
@@ -369,9 +341,6 @@ def fields(self) -> Tuple[str, ...]:
369341
client_id_target(table="eng_workflow_stable.build_v1"),
370342
# other
371343
DeleteTarget(table="telemetry_stable.pioneer_study_v4", field=PIONEER_ID),
372-
DeleteTarget(
373-
table="telemetry_stable.pre_account_v4", field=DESKTOP_ECOSYSTEM_CLIENT_ID
374-
),
375344
]
376345
}
377346

dags.yaml

-12
Original file line numberDiff line numberDiff line change
@@ -271,18 +271,6 @@ bqetl_fenix_event_rollup:
271271
retries: 2
272272
retry_delay: 30m
273273

274-
bqetl_account_ecosystem:
275-
schedule_interval: 0 2 * * *
276-
description: >-
277-
Tables related to the v1 implementation of Account Ecosystem Telemetry (AET)
278-
which is currently on hold.
279-
default_args:
280-
owner: jklukas@mozilla.com
281-
start_date: "2020-09-17"
282-
email: ["jklukas@mozilla.com"]
283-
retries: 2
284-
retry_delay: 30m
285-
286274
bqetl_stripe:
287275
schedule_interval: daily
288276
default_args:

dags/bqetl_account_ecosystem.py

-137
This file was deleted.

sql/moz-fx-data-shared-prod/account_ecosystem/aet_clients_daily/metadata.yaml

-16
This file was deleted.

sql/moz-fx-data-shared-prod/account_ecosystem/aet_clients_daily/view.sql

-61
This file was deleted.

sql/moz-fx-data-shared-prod/account_ecosystem_derived/desktop_clients_daily_v1/init.sql

-11
This file was deleted.

sql/moz-fx-data-shared-prod/account_ecosystem_derived/desktop_clients_daily_v1/metadata.yaml

-17
This file was deleted.

0 commit comments

Comments
 (0)