remove legacy table updates

mozilla / docker-etl

Collection of dockerized ETL jobs managed by data engineering.

Mozilla Public License 2.0

16 stars 14 forks source link

remove legacy table updates #245

Open jaredsnyder opened 2 months ago

jaredsnyder commented 2 months ago

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)
[ ] Scan the PR and verify that no changes (particularly to .circleci/config.yml) will cause environment variables (particularly credentials) to be exposed in test logs
[ ] Ensure the container image will be using permissions granted to telemetry-airflow responsibly.

jaredsnyder commented 2 months ago

Story: https://mozilla-hub.atlassian.net/browse/DENG-4383

m-d-bowerman commented 1 month ago

Looks good. Merging this is blocked by a view to create a replacement table, correct?

jaredsnyder commented 1 month ago

Looks good. Merging this is blocked by a view to create a replacement table, correct?

@m-d-bowerman Yeah I need to work with Sydney and Sergio to test the view from the notebook, it should be the same except the difference in where it starts so maybe I'm overthinking it