Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revisit materialized views #26

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

Revisit materialized views #26

wants to merge 1 commit into from

Conversation

jtcohen6
Copy link
Collaborator

Substantive changes

  • Resolve Snowflake materialized view fails on 2nd run #14 — Snowflake's caching logic can't differentiate between materialized views and views, so we just need to document that switching between view and materialized_view is unsupported
  • Fix auto_refresh logic on Redshift
  • Stick dispatched macros in the dbt macro namespace, rather than dbt_labs_materialized_views — this is a judgment call, to more closely mock the experience of these macros living "inside" dbt-core

etc

  • Revise README, including recommendations for git + subdirectory package installation and macro dispatch
  • Upgrade dbt-utils dependency

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Snowflake materialized view fails on 2nd run
1 participant