Skip to content

Commit 79eec08

Browse files
Update clone-incremental-models.md
1 parent 9573f2d commit 79eec08

File tree

1 file changed

+1
-8
lines changed

1 file changed

+1
-8
lines changed

website/docs/best-practices/clone-incremental-models.md

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -47,13 +47,6 @@ Because of your first clone step, the incremental models selected in your `dbt b
4747

4848
Your CI jobs will run faster, and you're more accurately mimicking the behavior of what will happen once the PR has been merged into main.
4949

50-
## Additional help
51-
52-
**Relevant `dbt clone` Slack thread:** https://dbt-labs.slack.com/archives/C05FWBP9X1U/p1692830261651829
53-
54-
### From the "Better CI for better data quality" coalesce talk
55-
"If you use the incremental materialization in your dbt project, you should consider cloning your relevant, pre-existing incremental models into your PR-specific schema as the first step of your CI check. This will force your second step to run in incremental mode (where is_incremental is true) because now the models already exist in your PR-specific schema (via cloning). This is beneficial because it more accurately mimics what will happen when you merge your changes into production and it will save time and money by not rebuilding your incremental models (which are often large data sets) from scratch for every PR that modifies them." -Grace Goheen, dbt Labs Product Manager
56-
5750
### Expansion on "think schema drift" where [on_schema_change](/docs/build/incremental-models#what-if-the-columns-of-my-incremental-model-change) config is set to `fail`" from above
5851

5952
Imagine you have an incremental model `my_incremental_model` with the following config:
@@ -83,4 +76,4 @@ There’s probably no perfect solution here; it’s all just tradeoffs! Our pref
8376
### Expansion on "why `state:old`"
8477

8578
For brand new incremental models, you want them to run in `full-refresh` mode in CI, because they will run in `full-refresh` mode in production when the PR is merged into `main`. They also don't exist yet in the production environment... they're brand new!
86-
If you don't specify this, you won't get an error just a “No relation found in state manifest for…”. So, it technically works without specifying `state:old` but adding `state:old` is more explicit and means it won't even try to clone the brand new incremental models.
79+
If you don't specify this, you won't get an error just a “No relation found in state manifest for…”. So, it technically works without specifying `state:old` but adding `state:old` is more explicit and means it won't even try to clone the brand new incremental models.

0 commit comments

Comments
 (0)