Merged PR 2498: Materialise int_dates_mtd and int_dates_by_deal as table to improve performance

# Description

Materialise int_dates_mtd and int_dates_by_deal as tables. This should improve the run speed as seen in local by quite a bit, and hopefully provide a better starting point for adding new dimensionality on business kpis.

I also documented these 2 models, that were missing :)

# Checklist

- [X] The edited models and dependants run properly with production data.
- [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

- [] Check if a full-refresh is required after this PR is merged.

Related work items: #19514
This commit is contained in:
Oriol Roqué Paniagua 2024-08-06 15:03:32 +00:00
parent b1bdbc0f39
commit 94bdc53adf
3 changed files with 137 additions and 15 deletions

View file

@ -1,7 +1,6 @@
/*
This model provides the necessary dates for each deal for deal-based KPIs models to work.
*/
{{ config(materialized="table", unique_key=["date", "id_deal"]) }}
with
int_dates as (select * from {{ ref("int_dates") }}),
int_core__unified_user as (select * from {{ ref("int_core__unified_user") }})