-
Notifications
You must be signed in to change notification settings - Fork 989
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
Updating description fields #6852
base: current
Are you sure you want to change the base?
Changes from 7 commits
bd7848d
b695a94
a0666d1
ad9fdec
accf0c1
8f132d6
3a610be
9bfdf52
f953800
3978555
3ae904d
719195d
f443914
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change | ||||||||
---|---|---|---|---|---|---|---|---|---|---|
@@ -1,7 +1,7 @@ | ||||||||||
--- | ||||||||||
title: "About incremental strategy" | ||||||||||
sidebar_label: "About incremental strategy" | ||||||||||
description: "Learn about the various ways (strategies) to implement incremental materializations." | ||||||||||
description: "Implement incremental materializations and the supported dbt adapters." | ||||||||||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What do you think about talking about "optimization" and talking about the new/changed data, which is key to incremental strategy.
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thinking about how we can shorten this. How about
Suggested change
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @matthewshaver I like that rewrite but wonder if we should include "materializations". in the title as a good key word. I think Joey mentioned the limit is 150 so we should be good? |
||||||||||
id: "incremental-strategy" | ||||||||||
--- | ||||||||||
|
||||||||||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,6 @@ | ||
--- | ||
title: "Project variables" | ||
description: "Use dbt project variables to configure conditional or reusable logic across models and other resources." | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 🔥 |
||
id: "project-variables" | ||
pagination_next: "docs/build/environment-variables" | ||
--- | ||
|
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.
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.
this reads great! you can use aliases for models, but other resources as well (seeds, snapshots, etc) so i'd enhance Leona's sugg as: