Merged PR 3909: Fixes created_date_utc and updated_date_utc in SH User and its dependencies
# Description The model `stg_core__superhog_user` contained the fields `created_date_utc` and `updated_date_utc`, which were actually timestamps, not dates. This created issues a while ago in KPIs that were fixed by forcing a date conversion at `int_kpis` level, but the source issue remained unfixed. This PR just creates the proper fields in staging, meaning: * `created_at_utc` and `created_date_utc` * `updated_at_utc` and `updated_date_utc` And propagates the changes through all dependencies. On KPIs, it also removes the hardcoded date conversion since it's no longer needed. It also adds the different schema entries on which these fields were already defined. Note I didn't update the staging entry since there's no entry for the rest of the fields. I recommend checking this PR in the commits order. What I've tested: * Unified User and User Host have plenty of dependencies that I've checked one by one, at each dependency layer by using dbt docs. * Run `core__unified_user` and the full execution of +`mtd_aggregated_metrics` What clearly will fail: * There's the famous `Users_dashboard` PBI report that shouldn't exist that has a dependency with `core__unified_user`. This report will fail. But it does not exist. I'd like to drop it once and forever # Checklist - [X] The edited models and dependants run properly with production data. **See above** - [X] The edited models are sufficiently documented. - [X] The edited models contain PK tests, and I've ran and passed them. - [X] I have checked for DRY opportunities with other models and docs. - [X] I've picked the right materialization for the affected models. # Other - [X] Check if a full-refresh is required after this PR is merged. **The staging model is incremental!** Related work items: #23703
This commit is contained in:
parent
803c5ed3d5
commit
d26e08bffd
8 changed files with 42 additions and 17 deletions
|
|
@ -13,8 +13,10 @@ with
|
|||
{{ adapter.quote("Avatar") }} as avatar,
|
||||
{{ adapter.quote("AirbnbId") }} as id_airbnb,
|
||||
{{ adapter.quote("AirbnbUrl") }} as airbnb_url,
|
||||
{{ adapter.quote("CreatedDate") }} as created_date_utc,
|
||||
{{ adapter.quote("UpdatedDate") }} as updated_date_utc,
|
||||
{{ adapter.quote("CreatedDate") }} as created_at_utc,
|
||||
date({{ adapter.quote("CreatedDate") }}) as created_date_utc,
|
||||
{{ adapter.quote("UpdatedDate") }} as updated_at_utc,
|
||||
date({{ adapter.quote("UpdatedDate") }}) as updated_date_utc,
|
||||
{{ adapter.quote("VerifiedDate") }} as verified_at_utc,
|
||||
cast({{ adapter.quote("VerifiedDate") }} as date) as verified_date_utc,
|
||||
{{ adapter.quote("FlagAsProblem") }} as flag_as_problem,
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue