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

feat: add doc for full_refresh option to on_schema_change #2546

Closed
1 task
pol-defont-reaulx opened this issue Dec 9, 2022 · 1 comment
Closed
1 task
Labels
blocked_by_dev Awaiting merge of PR with associated functionality content Improvements or additions to content icebox For issues we're closing but will revisit at a future date if possible! incremental models

Comments

@pol-defont-reaulx
Copy link

Contributions

  • I am a PM or subject matter expert at dbt who is responsible for this feature.

Where does this content belong?

I created the PR #6412 on dbt-core to add the full_refresh option to on_schema_change config of incremental models. This will allow the user to define that, when a schema change occurs on an incremental model, dbt needs to do a full refresh of this model. If there is no change in the model, then the full_refresh will not be activated.

This option need to be added to the section "What if the columns of my incremental model change?" of the incremental model page.

Link to source material

No source material

@pol-defont-reaulx pol-defont-reaulx added content Improvements or additions to content idea Proposes an idea for new content labels Dec 9, 2022
@runleonarun runleonarun added the dbt-core v1.4 Docs impact for the v1.4 release (Jan 2023) label Dec 16, 2022
@jtcohen6 jtcohen6 added the blocked_by_dev Awaiting merge of PR with associated functionality label Jan 2, 2023
@jtcohen6 jtcohen6 removed the dbt-core v1.4 Docs impact for the v1.4 release (Jan 2023) label Jan 10, 2023
@runleonarun runleonarun removed the idea Proposes an idea for new content label Mar 13, 2024
@mirnawong1
Copy link
Contributor

hey @pol-defont-reaulx thanks for opening this docs issue! I'm going to add an icebox label as it seems the original core pr dbt-labs/dbt-core#6412 was closed due to the decoupling. we can reopen this again once it goes through. really appreciate you opening this up!

@mirnawong1 mirnawong1 added the icebox For issues we're closing but will revisit at a future date if possible! label Nov 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
blocked_by_dev Awaiting merge of PR with associated functionality content Improvements or additions to content icebox For issues we're closing but will revisit at a future date if possible! incremental models
Projects
None yet
Development

No branches or pull requests

5 participants