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

add discourse link + simplify language #4675

Merged
merged 3 commits into from
Dec 19, 2023
Merged

Conversation

mirnawong1
Copy link
Contributor

@mirnawong1 mirnawong1 commented Dec 19, 2023

after discussion with @saraleon1 -- this pr adds a discourse link to incremental strategies discussion for large datasets and simplifies the 'when should i use an incremental model' paragraph/section. the discourse link has been share with users a few times and Sara suggested it'd be helpful to have this on the docs to help users.

Current content section

Screenshot 2023-12-19 at 13 06 23

new proposed content + link

  • Discourse article

  • In addition to these considerations for incremental models, it's important to understand their limits and challenges, particularly with large datasets. For more insights into efficient strategies, performance considerations, and the handling of late-arriving data in incremental models, refer to the On the Limits of Incrementality discourse discussion.

this pr adds a discourse link to incremental strategies discussion for large datasets and simplifies the 'when should i use an incremental model' paragraph/section.
@mirnawong1 mirnawong1 requested a review from a team as a code owner December 19, 2023 18:06
Copy link

vercel bot commented Dec 19, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-getdbt-com ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 19, 2023 6:31pm

@github-actions github-actions bot added content Improvements or additions to content Docs team Authored by the Docs team @dbt Labs size: small This change will take 1 to 2 days to address and removed Docs team Authored by the Docs team @dbt Labs labels Dec 19, 2023
@mirnawong1 mirnawong1 enabled auto-merge December 19, 2023 18:26
@mirnawong1 mirnawong1 merged commit c5e0f60 into current Dec 19, 2023
7 checks passed
@mirnawong1 mirnawong1 deleted the mirnawong1-patch-22 branch December 19, 2023 18:31
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 Docs team Authored by the Docs team @dbt Labs size: small This change will take 1 to 2 days to address
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants