From 873402fd8e3257e5fc8b79f24679da446c096f84 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Oriol=20Roqu=C3=A9=20Paniagua?= Date: Sun, 19 Jan 2025 09:37:07 +0000 Subject: [PATCH] Merged PR 4105: Fixes exclusion of MRR after name change # Description After the name change of the metric, the exclusion in reporting for the ongoing month and previous month is not working for the new MRR metric. This PR aims to fix it. # Checklist - [ ] The edited models and dependants run properly with production data. - [ ] The edited models are sufficiently documented. - [ ] The edited models contain PK tests, and I've ran and passed them. - [ ] I have checked for DRY opportunities with other models and docs. - [ ] I've picked the right materialization for the affected models. # Other - [ ] Check if a full-refresh is required after this PR is merged. Fixes exclusion of MRR after name change Related work items: #26621 --- models/reporting/general/mtd_aggregated_metrics.sql | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/models/reporting/general/mtd_aggregated_metrics.sql b/models/reporting/general/mtd_aggregated_metrics.sql index bdfad23..686a469 100644 --- a/models/reporting/general/mtd_aggregated_metrics.sql +++ b/models/reporting/general/mtd_aggregated_metrics.sql @@ -68,7 +68,7 @@ where or lower(metric) like '%resolutions%' or lower(metric) like '%invoiced%' or lower(metric) like '%retained%' - or lower(metric) like '%expected mrr%' + or lower(metric) like '%mrr%' or lower(metric) like '%damage host%' ) and {{ is_date_before_previous_month("date") }} @@ -80,7 +80,7 @@ where or lower(metric) like '%resolutions%' or lower(metric) like '%invoiced%' or lower(metric) like '%retained%' - or lower(metric) like '%expected mrr%' + or lower(metric) like '%mrr%' or lower(metric) like '%damage host%' ) )