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

feat(kevel-metadata): adding new etl for extracting kevel metadata from kevel api into ads_derived table #242

Draft
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

braunreyes
Copy link

@braunreyes braunreyes commented Jul 19, 2024

References https://mozilla-hub.atlassian.net/browse/AD-295

This is a new job that will extract ad data from the kevel API and load into a raw data table called `ads_derived.kevel_metadata_history_v1.

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.

TO DO:
Finalize the merge logic to comply with stakeholder expectations. Right now the daily partition is overwritten on each run. @sergiosonline @curtismorales you will want to discuss how you want to handle things like 'what if ad_id is in a different flight with different attributes in the same day?'.

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.

1 participant