-
Notifications
You must be signed in to change notification settings - Fork 969
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
[merge after dec 5th] adds new hard_deletes config #6558
Conversation
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
| 1 | b1885d098f8bcff53... | shipped | 2024-10-02 ... | 2024-06-03 ... | | False | | ||
| 2 | b1885d098f8bcff55... | active | 2024-10-02 ... | 2024-05-19 ... | | False | | ||
|
||
In this example, the `dbt_is_deleted` column is set to `True` when the record is deleted. When the record is restored, the `dbt_is_deleted` column is set to `False`. |
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.
would love a gut check if this is right bc I've been reading this over and over again so might have mixed things up.
website/docs/reference/resource-configs/snapshot_meta_column_names.md
Outdated
Show resolved
Hide resolved
| -- | -------------------- | -------------------- | -------------------- | -------------------- | | ||
| 1 | 60a1f1dbdf899a4dd... | 2024-10-02 ... | 2024-10-02 ... | 2024-10-02 ... | | ||
| 2 | b1885d098f8bcff51... | 2024-10-02 ... | 2024-10-02 ... | | | ||
| id | scd_id | modified_date | start_date | end_date | is_deleted | |
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.
same comment as earlier
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.
i changed it to the below but also wasn't sure if it made sense to go into hard_deletes:'new_record'
detail since this page is about meta column names?
id | scd_id | modified_date | start_date | end_date | is_deleted |
---|---|---|---|---|---|
1 | 60a1f1dbdf899a4dd... | 2024-10-02 ... | 2024-10-02 ... | 2024-10-03 ... | False |
1 | 60a1f1dbdf899a4dd... | 2024-10-03 ... | 2024-10-03 ... | True | |
2 | b1885d098f8bcff51... | 2024-10-02 ... | 2024-10-02 ... | False |
Co-authored-by: Grace Goheen <[email protected]>
Co-authored-by: Grace Goheen <[email protected]>
Co-authored-by: Grace Goheen <[email protected]>
Co-authored-by: Grace Goheen <[email protected]>
Co-authored-by: Grace Goheen <[email protected]>
…ames.md Co-authored-by: Grace Goheen <[email protected]>
Co-authored-by: Grace Goheen <[email protected]>
Co-authored-by: Grace Goheen <[email protected]>
Co-authored-by: Grace Goheen <[email protected]>
Co-authored-by: Grace Goheen <[email protected]>
Co-authored-by: Grace Goheen <[email protected]>
Co-authored-by: Grace Goheen <[email protected]>
Co-authored-by: Grace Goheen <[email protected]>
Co-authored-by: Grace Goheen <[email protected]>
….9.md Co-authored-by: Grace Goheen <[email protected]>
Co-authored-by: Grace Goheen <[email protected]>
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.
website/docs/reference/resource-configs/invalidate_hard_deletes.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Doug Beatty <[email protected]>
Co-authored-by: Doug Beatty <[email protected]>
Co-authored-by: Doug Beatty <[email protected]>
Co-authored-by: Doug Beatty <[email protected]>
hey @dbeatty10 , woudl this work for the check strategy table example? Snapshot results with
|
folded in changes and approved by doug
Check your created Crawler Check your created index on your Algolia Application |
this pr adds the new
hard_deletes
config, which replacesinvalidates_hard_deletes
.added in multiple pages such as snapshots, snapshot config, new hard_deletes page, upgrade guide, and release note.
This needs to be merged after Dec 5th
Resolves #6522
🚀 Deployment available! Here are the direct links to the updated files: