-
Notifications
You must be signed in to change notification settings - Fork 987
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 new spec to snapshot page #6188
add new spec to snapshot page #6188
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
One non-blocking sugg
website/docs/docs/build/snapshots.md
Outdated
```yaml | ||
-- models/ephemeral_orders.sql | ||
{{ config(materialized='ephemeral') }} |
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.
For consistency with the other examples, I'd suggest using <File ...>...</File>
for the file named models/ephemeral_orders.sql
and then remove the comment containing the file name.
`
|
||
<File name='models/changed_orders.sql'> | ||
If you need to clean or transform your data before snapshotting, create an ephemeral model (or a staging model) that applies the necessary transformations. Then, reference this model in your snapshot configuration. This approach keeps your snapshot definitions clean and allows you to test and run transformations separately. |
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 pr contais the following:
this pr should merge into the main
add/new-snapshot-spec-main
branch