feature: Metric views #1285
Open
feature: Metric views #1285
Conversation
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
### Description I don't know how this was happening, since the code for change detection seems to not return a TagsConfig when I go into via debug, but we were calling alter with an empty set tags set, which was raising exceptions. ### Checklist - [x] I have run this code in development and it appears to resolve the stated issue - [x] This PR includes tests, or tests are not required/relevant for this PR - [ ] I have updated the `CHANGELOG.md` and added information about my change to the "dbt-databricks next" section.
### Description Migrating to SDK for every REST call ### Checklist - [x] I have run this code in development and it appears to resolve the stated issue - [x] This PR includes tests, or tests are not required/relevant for this PR - [x] I have updated the `CHANGELOG.md` and added information about my change to the "dbt-databricks next" section. --------- Co-authored-by: Eric Jang <eric.jang@databricks.com>
Resolved merge conflicts and fixed metric view issues: - tags.sql: Restored metric view handling (ALTER METRIC VIEW is invalid SQL, must use ALTER VIEW for metric views) - replace.sql: Fixed to check target_relation.is_metric_view since metric views are stored as VIEW type in information_schema - metric_view.sql: Simplified to always use CREATE OR REPLACE to avoid type mismatch issues between ViewConfig and MetricViewConfig - relation.py: Merged metric view support with iceberg can_be_replaced support All unit tests (710) and metric view functional tests (11) pass.
TestDebugMetricViewChanges was a debugging artifact: - Named 'Debug' indicating temporary purpose - Functionality already covered by TestMetricViewSimpleChanges - view_update_via_alter config no longer changes behavior after simplification
Coverage report
Click to see where and how coverage changed
This report was generated by python-coverage-comment-action |
||||||||||||||||||||||||||||||||||||||||||
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
Adds initial implementation of UC metric views as a materialization
Checklist
CHANGELOG.mdand added information about my change to the "dbt-databricks next" section.