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

New materialized view adapter support #3593

Merged
merged 6 commits into from
Jun 29, 2023
Merged

New materialized view adapter support #3593

merged 6 commits into from
Jun 29, 2023

Conversation

matthewshaver
Copy link
Contributor

What are you changing in this pull request and why?

Documenting support for materialize_view on adapters for:
Bigquery
Redshift
Postgres

Checklist

  • Review the Content style guide and About versioning so my content adheres to these guidelines.
  • Add a checklist item for anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."

@matthewshaver matthewshaver requested review from a team and dataders as code owners June 21, 2023 16:02
@netlify
Copy link

netlify bot commented Jun 21, 2023

Deploy Preview for docs-getdbt-com ready!

Name Link
🔨 Latest commit f69100a
🔍 Latest deploy log https://app.netlify.com/sites/docs-getdbt-com/deploys/649ccae697953f000816700f
😎 Deploy Preview https://deploy-preview-3593--docs-getdbt-com.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@github-actions github-actions bot added content Improvements or additions to content Docs team Authored by the Docs team @dbt Labs size: medium This change will take up to a week to address labels Jun 21, 2023
@matthewshaver matthewshaver linked an issue Jun 21, 2023 that may be closed by this pull request
3 tasks
@mikealfare mikealfare self-assigned this Jun 21, 2023
Copy link
Contributor

@dataders dataders left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

two high-level questions. is it the scope of this PR to:

  • document Snowflake DT support?
  • add a page to the "Build: Enhance your models" akin to the Incremental Models page?
image

Copy link
Collaborator

@runleonarun runleonarun left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just one small feedback comment: can you create a reusable for the example? It seems the same in all three. Here's what that would look like: https://github.com/dbt-labs/docs.getdbt.com/pull/3616/files

- `skip` — allows runs to continue while also providing a warning that the model was skipped
- `fail` — forces runs to fail if a change is detected in a materialized view

You can create a materialized view by editing _one_ of these files:
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matthewshaver can we add line 779 - 806 into a partial (was snippet) and reuse this content for all three pages? This will help us maintain docs content and more update this section in the future. I think these lines are exactly the same, but I didn't do a diff, so 🤷🏻‍♀️

@matthewshaver matthewshaver merged commit 5a041e3 into current Jun 29, 2023
@matthewshaver matthewshaver deleted the materialized branch June 29, 2023 00:29
jtcohen6 added a commit that referenced this pull request Jul 18, 2023
resolves #3696
resolves #3723

Revise v1.6 migration guide for readability. Still a handful of TODOs.

### Materialized views
- Add `VersionBlock` components for adapter MVs, missed in
#3593
- We seem to be missing docs on Snowflake dynamic tables. I don't know
why #3494 was closed.

### Other updates
- Add manifest v10 (will need addition to schemas.getdbt.com:
dbt-labs/schemas.getdbt.com#24)
- Add note to upgrade guide for `state:old` and `state:unmodified`
(missed in #3596). I
haven't done a thorough comb-through of other closed v1.6 PRs to see
what needs addition to the migration guide.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Improvements or additions to content Docs team Authored by the Docs team @dbt Labs size: medium This change will take up to a week to address
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Materialized Views
4 participants