Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove amplitude views #2279

Merged
merged 5 commits into from
Aug 23, 2021
Merged

Remove amplitude views #2279

merged 5 commits into from
Aug 23, 2021

Conversation

jklukas
Copy link
Contributor

@jklukas jklukas commented Aug 20, 2021

We never cleaned these up after removing the Amplitude export tasks in
mozilla/telemetry-airflow#1328

Checklist for reviewer:

  • Commits should reference a bug or github issue, if relevant (if a bug is referenced, the pull request should include the bug number in the title)
  • If the PR comes from a fork, trigger integration CI tests by running the Push to upstream workflow and provide the <username>:<branch> of the fork as parameter. The parameter will also show up
    in the logs of the manual-trigger-required-for-fork CI task together with more detailed instructions.
  • If adding a new field to a query, ensure that the schema and dependent downstream schemas have been updated

We never cleaned these up after removing the Amplitude export tasks in
mozilla/telemetry-airflow#1328
@jklukas jklukas requested a review from relud August 20, 2021 19:25
@jklukas
Copy link
Contributor Author

jklukas commented Aug 20, 2021

We will need to manually delete the prod views once this is merged.

@jklukas jklukas merged commit 53e9a31 into main Aug 23, 2021
@jklukas jklukas deleted the delete-amplitude-views branch August 23, 2021 18:27
@jklukas
Copy link
Contributor Author

jklukas commented Aug 23, 2021

Ran the following to clean up views:

bq rm moz-fx-data-shared-prod:firefox_accounts.fxa_amplitude_export
bq rm moz-fx-data-shared-prod:firefox_accounts.sync_send_tab_export
bq rm moz-fx-data-shared-prod:messaging_system.onboarding_events_amplitude
bq rm moz-fx-data-shared-prod:messaging_system.onboarding_retention_events_amplitude
bq rm moz-fx-data-shared-prod:telemetry_derived.devtools_events_amplitude_v1
bq rm mozdata:firefox_accounts.fxa_amplitude_export
bq rm mozdata:firefox_accounts.sync_send_tab_export
bq rm mozdata:messaging_system.onboarding_events_amplitude
bq rm mozdata:messaging_system.onboarding_retention_events_amplitude

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants