-
Notifications
You must be signed in to change notification settings - Fork 991
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
Deprecated environment variables for state and defer #6784
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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 @dbeatty10 ! great catches and thank you for opening this up to clarify it further!
@@ -157,6 +148,12 @@ If both the flag and env var are provided, the flag takes precedence. | |||
- The `--state` artifacts must be of schema versions that are compatible with the currently running dbt version. | |||
- These are powerful, complex features. Read about [known caveats and limitations](/reference/node-selection/state-comparison-caveats) to state comparison. | |||
|
|||
:::warning Syntax deprecated | |||
|
|||
In [dbt v1.5](/docs/dbt-versions/core-upgrade/upgrading-to-v1.5#behavior-changes), we deprecated the original syntax for state (`DBT_ARTIFACT_STATE_PATH`) and defer (`DBT_DEFER_TO_STATE`). Although dbt supports backward compatibility with the old syntax, we will remove it in a future release that we have not yet determined. |
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.
[custom.Typos] Oops there's a typo -- did you really mean 'v1.5'?
What are you changing in this pull request and why?
In dbt v1.5, we deprecated the original syntax for state and defer:
DBT_ARTIFACT_STATE_PATH
->DBT_STATE
DBT_DEFER_TO_STATE
->DBT_DEFER
We want to make that deprecation clear without reducing the readability of the current syntax, so:
DBT_ARTIFACT_STATE_PATH
withDBT_STATE
Noticed these while responding to dbt-labs/dbt-core#11216.
Checklist
🚀 Deployment available! Here are the direct links to the updated files: