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

stage MITx Online tracking log table, update to xPro staging tracking log #792

Merged
merged 1 commit into from
Aug 10, 2023

Conversation

rachellougee
Copy link
Contributor

@rachellougee rachellougee commented Aug 8, 2023

What are the relevant tickets?

#729

Description (What does it do?)

This adds stg__mitxonline__openedx__tracking_logs__user_activity as an incremental model for performance. It has dedupe logic because raw data contains duplicates. It also updates to stg__mitxpro__openedx__tracking_logs__user_activity to be consistent with mitxonline.

How can this be tested?

To test against production data, you will need to update your profile.xml with production host mitol-ol-data-lake-production.trino.galaxy.starburst.io due to resource limitation of our QA cluster to run large model (#793 is created to solve this problem without updating profiles.yml)
then run

 dbt build --select stg__mitxonline__openedx__tracking_logs__user_activity --vars 'schema_suffix: YOURSCHEMA' --target dev_production 

 dbt build --select stg__mitxpro__openedx__tracking_logs__user_activity --vars 'schema_suffix: YOURSCHEMA' --target dev_production --full-refresh

Tests should all pass

@rachellougee rachellougee linked an issue Aug 8, 2023 that may be closed by this pull request
@abeglova abeglova self-assigned this Aug 10, 2023
Copy link
Contributor

@abeglova abeglova left a comment

Choose a reason for hiding this comment

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

LGTM

@@ -0,0 +1,67 @@
-- xPro user activities from tracking logs
Copy link
Contributor

Choose a reason for hiding this comment

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

xPro -> mitxonline

@rachellougee rachellougee force-pushed the 729-explore-and-stage-mitx-online-tracking-logs branch from 7261ca0 to 43bb0f5 Compare August 10, 2023 17:51
@rachellougee rachellougee merged commit af9eeab into main Aug 10, 2023
@rachellougee rachellougee deleted the 729-explore-and-stage-mitx-online-tracking-logs branch August 10, 2023 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

stage MITx Online tracking logs
2 participants