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.
Description & motivation
Adding cohortYear in
dim_student
. This is from a collection (relevant staging PR that flattens this).Relevant Monday item
PR Merge Priority:
Changes to existing files:
dim_student
: just adding cohort year build model and joining itedu_wh/models/core_warehouse/dim_student.sql
Lines 220 to 222 in 08f6872
New files created:
bld_ef3__student_cohort_years
: to bring back to k_student grain after flattening the collectionTests and QC done:
This branch has been tested in Boston and SC-district warehouse.
Future ToDos
No other collection in stuEdOrg needed to be flatten, there were other fields but non of them are populated in either Boston or SC yet. But in the future, could add those in as well