-
Notifications
You must be signed in to change notification settings - Fork 978
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 behavior changes: state_modified_compare_more_unrendered
and skip_nodes_if_on_run_start_fails
#6204
Merged
Merged
New behavior changes: state_modified_compare_more_unrendered
and skip_nodes_if_on_run_start_fails
#6204
Changes from 2 commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
f68f723
on run start behavior change
matthewshaver 66f612f
adding state:modified
matthewshaver adb5fa2
Adding parameter to table
matthewshaver 316b53b
Merge branch 'current' into hook-change
runleonarun 488e1f5
Merge branch 'current' into hook-change
mirnawong1 97430a2
Merge branch 'current' into hook-change
runleonarun 0b2f1db
New updates
matthewshaver 257bc4a
Undoing change
matthewshaver 53b4d15
Removing content
matthewshaver ebe1bb1
Merge branch 'current' into hook-change
matthewshaver b493b5c
Splitting release note
matthewshaver 1b34dbe
Merge branch 'hook-change' of https://github.com/dbt-labs/docs.getdbt…
matthewshaver 4054384
Moving new content to top
matthewshaver 789e4e5
Update website/docs/docs/dbt-versions/release-notes.md
matthewshaver 44dd028
Removing until `state_modified_compare_vars` is shipped and available
dbeatty10 3940c63
Update website/docs/reference/global-configs/behavior-changes.md
matthewshaver 5307d86
Merge branch 'current' into hook-change
matthewshaver b1ad4e0
Keep the `restrict_direct_pg_catalog_access` row as-is
dbeatty10 2cfa8f2
Update website/docs/reference/global-configs/behavior-changes.md
matthewshaver 027e6d2
Apply suggestions from code review
matthewshaver b1f29f0
Update website/docs/reference/global-configs/behavior-changes.md
matthewshaver 07c74a6
Update website/docs/reference/global-configs/behavior-changes.md
matthewshaver File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -68,6 +68,8 @@ When we use dbt Cloud in the following table, we're referring to accounts that h | |
| require_resource_names_without_spaces | 2024.05.146 | TBD* | 1.8.0 | 1.9.0 | | ||
| source_freshness_run_project_hooks | 2024.03.61 | TBD* | 1.8.0 | 1.9.0 | | ||
| [Redshift] [restrict_direct_pg_catalog_access](#redshift-restrict_direct_pg_catalog_access) | 2024.09.242 | TBD* | dbt-redshift v1.9.0 | 1.9.0 | | ||
| skip_nodes_if_on_run_start_fails | | TBD* | 1.9.0 | TBD* | | ||
| state_modified_compare_more_unrendered | | TBD* | 1.9.0 | TBD* | | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @jtcohen6 and @graciegoheen same for the state modified changes |
||
|
||
When the dbt Cloud Maturity is "TBD," it means we have not yet determined the exact date when these flags' default values will change. Affected users will see deprecation warnings in the meantime, and they will receive emails providing advance warning ahead of the maturity date. In the meantime, if you are seeing a deprecation warning, you can either: | ||
- Migrate your project to support the new behavior, and then set the flag to `True` to stop seeing the warnings. | ||
|
@@ -129,6 +131,20 @@ on-run-start: | |
``` | ||
</File> | ||
|
||
### On-run-start hook | ||
|
||
The flag is `False` by default. | ||
|
||
Set the `skip_nodes_if_on_run_start_fails` flag to `True` to skip all selected resources from running if there is a failure on an `on-run-start` hook. | ||
|
||
### Source definitions and variables for state:modified | ||
|
||
The flags are `False` by default. | ||
|
||
Set `state_modified_compare_more_unrendered` to `True` to start persisting `unrendered_database` and `unrendered_schema` configs during source parsing, and do comparison on unrendered values during `state:modified` checks. Setting the flag to `True` reduces false positives during `state:modified` checks when `prod` and `dev` environments have different configs. | ||
|
||
Set the `state_modified_compare_vars` to `True` if a model uses a `var` or `env_var` in its definition. dbt will be able to identify its lineage to include the model in `state:modified` because the `var` or `env_var` value has changed. | ||
|
||
## Adapter-specific behavior changes | ||
Some adapters may show behavior changes when certain flags are enabled. Refer to the following sections for each respective adapter. | ||
### [Redshift] restrict_direct_pg_catalog_access | ||
|
Oops, something went wrong.
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.
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.
@jtcohen6 or @graciegoheen, what should be the value for Cloud Intro here?