-
Notifications
You must be signed in to change notification settings - Fork 61
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
ADAP-389: Stub materialized view as a materialization #387
ADAP-389: Stub materialized view as a materialization #387
Conversation
…arsed macro into smaller pieces
…removed dbt-postgres dependency
…ase in `dbt-core`
Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the dbt-redshift contributing guide. |
I'm skipping the change log on this; it's too granular. There will be a changelog at the feature level that will be merged into |
…iew instead of view
…iew instead of view
…lized-views/ADAP-389
…lized-views/ADAP-389
…lized-views/ADAP-389
…lized-views/ADAP-389
…lized-views/ADAP-389 # Conflicts: # dbt/include/redshift/macros/adapters.sql
I'm running into an issue while trying to recreate materialized views. They don't appear to show up as expected in |
…lized-views/ADAP-389 # Conflicts: # dbt/include/redshift/macros/adapters.sql # setup.py
resolves dbt-labs/dbt-core#6911
Description
Stub out the materialized view materialization for Redshift. Under the hood this just creates a view for now. But this can be used to verify all of the plumbing.
While creating the materialization, it was observed that other materialization templates existed outside of the
materialization
directory. These were moved fromadapters.sql
into thematerialization
directory in the appropriate file (e.g.table.sql
). Some of the macros were also refactored to be smaller bits of code anddbt-postgres
dependencies were eliminated when found.Checklist
changie new
to create a changelog entry