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 export tasks #1328

Merged
merged 2 commits into from
Jul 6, 2021
Merged

Remove Amplitude export tasks #1328

merged 2 commits into from
Jul 6, 2021

Conversation

jklukas
Copy link
Contributor

@jklukas jklukas commented Jun 22, 2021

Noticed that these DAGs still exist while doing Airflow triage.

I need to get verification that these use cases are truly spun down.

Copy link
Contributor

@fbertsch fbertsch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We need to wait on this until the EOM. My plan was to disable at EOW, but we can keep this around to remove them after the wellness week.

@jklukas jklukas requested a review from fbertsch July 6, 2021 13:56
Copy link
Contributor

@fbertsch fbertsch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r+wc

GRAVEYARD.md Outdated Show resolved Hide resolved
jklukas and others added 2 commits July 6, 2021 11:36
Noticed that these DAGs still exist while doing Airflow triage.

I need to get verification that these use cases are truly spun down.
Co-authored-by: Frank Bertsch <fbertsch@mozilla.com>
@jklukas jklukas merged commit a166f61 into main Jul 6, 2021
@jklukas jklukas deleted the remove-amplitudee branch July 6, 2021 15:53
jklukas added a commit to mozilla/bigquery-etl that referenced this pull request Aug 20, 2021
We never cleaned these up after removing the Amplitude export tasks in
mozilla/telemetry-airflow#1328
jklukas added a commit to mozilla/bigquery-etl that referenced this pull request Aug 23, 2021
* Remove amplitude views

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

* Update graveyard doc

Co-authored-by: Daniel Thorn <dthorn@mozilla.com>
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