You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
⚠️ Breaking tag change — affects 4 models but has broader blast radius
The diff renames the finance tag to fin across orders, returned_orders, stg_orders, and stg_payments. However, Elementary currently shows 9 models carrying the finance tag — including artificial_anomaly, date_based_anomaly, returned_orders_anomaly, forced_anomaly, and return_rate_anomaly (all in models/schema.yml). Those 5 models are not updated in this PR, meaning after merge the codebase will have an inconsistent mix of finance and fin tags.
Risks:
Any alert routing rules, filters, or dashboards scoped to the finance tag will stop matching the 4 updated models immediately on merge.
The 5 unmodified models retain finance, so tag-based queries will return split/incomplete results.
If fin is a new tag (not an existing one), any downstream consumers expecting finance get nothing.
Recommendation: Either update all 9 finance-tagged models in this PR, or revert to keeping finance as-is. If this rename is intentional, it should be a single atomic change covering the whole project and any dependent alert/routing configurations.
Risk: Medium — No SQL logic or schema changes; purely metadata. Low breakage risk to pipelines, but tag-based filtering and alerting will be silently inconsistent after merge.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
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.
Elementary: This PR updates your dbt assets according to changes made in Elementary's UI.
Created by:
michael@elementary-data.com