-
Notifications
You must be signed in to change notification settings - Fork 977
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
feature: add new dynamic table paramaters #4835
feature: add new dynamic table paramaters #4835
Conversation
Hello!👋 Thanks for contributing to the dbt product documentation and opening this pull request! ✨ |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
@matthewshaver is attempting to deploy a commit to the dbt-labs Team on Vercel. A member of the Team first needs to authorize it. |
@HenkvanDyk and @dataders what is the status of this draft? Should we move forward here? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @matthewshaver I noticed they have a new PR for this work. Removing comment
per pr#1081
@@ -360,6 +360,10 @@ Dynamic tables are supported with the following configuration parameters: | |||
| [`on_configuration_change`](/reference/resource-configs/on_configuration_change) | `<string>` | no | `apply` | n/a | | |||
| [`target_lag`](#target-lag) | `<string>` | yes | | alter | | |||
| [`snowflake_warehouse`](#configuring-virtual-warehouses) | `<string>` | yes | | alter | | |||
| [`refresh_mode`](#refresh-mode) | `<string>` | no | | n/a | | |||
| [`initialize`](#initialize) | `<string>` | no | | n/a | | |||
| [`comment`](#comment) | `<string>` | no | | n/a | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Per dbt-labs/dbt-snowflake#1081 we removed comment
| [`comment`](#comment) | `<string>` | no | | n/a | |
|
||
Refreshes the dynamic table at the next scheduled refresh. | ||
|
||
### Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
per dbt-labs/dbt-snowflake#1081 removing comment
### Comment |
Refreshes the dynamic table at the next scheduled refresh. | ||
|
||
### Comment | ||
Specifies a comment for the dynamic table. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
per dbt-labs/dbt-snowflake#1081 removing comment
Specifies a comment for the dynamic table. |
docs added here: |
What are you changing in this pull request and why?
Documentation for features added here dbt-labs/dbt-snowflake#893
Checklist
Adding or removing pages (delete if not applicable):
website/sidebars.js
website/static/_redirects
npm run build
to update the links that point to deleted pages