Remove legacy per-app Glean usage ETL #895
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The incline dashboard, KPIs, and GUD are now referencing the generic glean ETL (baseline_clients_daily and baseline_clients_last_seen), so these other tables should no longer be needed. I also need to remove these from Airflow and I will need to send out a message to fx-data-dev to advertise this breaking change, but I doubt there's much usage so far of these legacy tables since they have not been well documented or advertised; they existed primarily to serve GUD, Incline, and KPIs.
jklukas
force-pushed
the
generic-glean-usage
branch
from
fd1103a to
e36c3be
Compare
fbertsch
approved these changes
Apr 13, 2020
I don't have proof of this, but I will send out an announcement with at least a few days for folks to report uses before I merge this and delete things.
The generic glean ETL is all new tables |
|
This was done in #925 instead. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The incline dashboard, KPIs, and GUD are now referencing the generic glean ETL
(baseline_clients_daily and baseline_clients_last_seen), so these other
tables should no longer be needed.
I also need to remove these from Airflow and I will need to send out a message
to fx-data-dev to advertise this breaking change, but I doubt there's much
usage so far of these legacy tables since they have not been well documented
or advertised; they existed primarily to serve GUD, Incline, and KPIs.