Skip to content

docs(snowflake): document transient tmp_relation_type and resolve macro - #9098

Merged
runleonarun merged 5 commits into
currentfrom
bper/snowflake-transient-tmp-relation-docs
May 13, 2026
Merged

docs(snowflake): document transient tmp_relation_type and resolve macro#9098
runleonarun merged 5 commits into
currentfrom
bper/snowflake-transient-tmp-relation-docs

Conversation

@b-per

@b-per b-per commented May 5, 2026

Copy link
Copy Markdown
Contributor

What are you changing in this pull request and why?

Documents the new transient option for tmp_relation_type in Snowflake incremental models, added in dbt-labs/dbt-adapters#1894.

Changes to website/docs/reference/resource-configs/snowflake-configs.md (Temporary tables section):

  • Adds transient as a valid value for tmp_relation_type alongside table and view
  • Adds a bullet-point comparison of all three options (default behavior, catalog visibility, lineage tracking, fail-safe trade-offs)
  • Adds a :::warning admonition about concurrent-run conflicts when using transient (since the tmp table persists in the catalog under a deterministic name)
  • Adds a new ### Avoiding tmp relation conflicts subsection documenting the snowflake__resolve_incremental_tmp_relation dispatch macro with two usage examples (simple scratch schema and CI-job-aware schema)

Checklist

@vercel

vercel Bot commented May 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
docs-getdbt-com Ready Ready Preview May 13, 2026 4:03pm

Request Review

@github-actions github-actions Bot added the content Improvements or additions to content label May 5, 2026
Adds `transient` as a valid value for `tmp_relation_type` in Snowflake
incremental models, explains lineage tracking benefits and fail-safe
trade-offs, adds a warning about concurrent-run conflicts, and documents
the `snowflake__resolve_incremental_tmp_relation` dispatch macro as the
recommended fix.

Tracks: dbt-labs/dbt-adapters#1894
@b-per
b-per force-pushed the bper/snowflake-transient-tmp-relation-docs branch from 8912c19 to c515978 Compare May 5, 2026 08:53
@b-per

b-per commented May 5, 2026

Copy link
Copy Markdown
Contributor Author

We should merge this when the new config is available (next release on the dbt platform)

@b-per
b-per marked this pull request as ready for review May 12, 2026 16:31
Copilot AI review requested due to automatic review settings May 12, 2026 16:32
@b-per
b-per requested a review from a team as a code owner May 12, 2026 16:32
@b-per

b-per commented May 12, 2026

Copy link
Copy Markdown
Contributor Author

The underlying feature has been released and is now available on the dbt platform — marking this as ready for review.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Snowflake adapter configuration docs to cover the new transient option for tmp_relation_type in incremental models, including guidance for avoiding tmp-relation naming conflicts during concurrent runs.

Changes:

  • Add transient as a documented tmp_relation_type value alongside view and table, with a comparison of trade-offs.
  • Add a warning callout about concurrent-run conflicts when using transient tmp relations.
  • Document how to override snowflake__resolve_incremental_tmp_relation to redirect tmp relations into an isolated schema.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread website/docs/reference/resource-configs/snowflake-configs.md
Comment thread website/docs/reference/resource-configs/snowflake-configs.md Outdated
Comment thread website/docs/reference/resource-configs/snowflake-configs.md Outdated

@runleonarun runleonarun left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@b-per just a few suggestions

Comment thread website/docs/reference/resource-configs/snowflake-configs.md Outdated
Comment thread website/docs/reference/resource-configs/snowflake-configs.md Outdated
Comment thread website/docs/reference/resource-configs/snowflake-configs.md Outdated
Comment thread website/docs/reference/resource-configs/snowflake-configs.md Outdated
Comment thread website/docs/reference/resource-configs/snowflake-configs.md Outdated
Comment thread website/docs/reference/resource-configs/snowflake-configs.md Outdated
Comment thread website/docs/reference/resource-configs/snowflake-configs.md Outdated
Comment thread website/docs/reference/resource-configs/snowflake-configs.md Outdated
Comment thread website/docs/reference/resource-configs/snowflake-configs.md Outdated
Co-authored-by: Leona B. Campbell <3880403+runleonarun@users.noreply.github.com>

@runleonarun runleonarun left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM I will check the staging then merge

Comment thread website/docs/reference/resource-configs/snowflake-configs.md Outdated
Comment thread website/docs/reference/resource-configs/snowflake-configs.md Outdated
@runleonarun
runleonarun merged commit 256e907 into current May 13, 2026
4 of 5 checks passed
@runleonarun
runleonarun deleted the bper/snowflake-transient-tmp-relation-docs branch May 13, 2026 16:48
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants