-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Alert section for freshness checks #21676
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
docs/content/concepts/assets/asset-checks/checking-for-data-freshness.mdx
Outdated
Show resolved
Hide resolved
docs/content/concepts/assets/asset-checks/checking-for-data-freshness.mdx
Outdated
Show resolved
Hide resolved
docs/content/concepts/assets/asset-checks/checking-for-data-freshness.mdx
Outdated
Show resolved
Hide resolved
docs/content/concepts/assets/asset-checks/checking-for-data-freshness.mdx
Outdated
Show resolved
Hide resolved
docs/content/concepts/assets/asset-checks/checking-for-data-freshness.mdx
Outdated
Show resolved
Hide resolved
docs/content/concepts/assets/asset-checks/checking-for-data-freshness.mdx
Outdated
Show resolved
Hide resolved
5b487f9
to
e316d9e
Compare
docs/content/concepts/assets/asset-checks/checking-for-data-freshness.mdx
Outdated
Show resolved
Hide resolved
docs/content/concepts/assets/asset-checks/checking-for-data-freshness.mdx
Outdated
Show resolved
Hide resolved
e316d9e
to
dd26305
Compare
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.
Should we instead add this section to the asset checks page and link to it from the freshness checks page?
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.
Two small things, but lgtm!
docs/content/concepts/assets/asset-checks/checking-for-data-freshness.mdx
Outdated
Show resolved
Hide resolved
docs/content/concepts/assets/asset-checks/checking-for-data-freshness.mdx
Outdated
Show resolved
Hide resolved
@sryza Which asset check page - this main category one, or this guide for defining checks? What's the thinking behind moving it? |
I honestly didn't think that far - I could see either, or even a third one.
These instructions aren't really specific to freshness checks in particular. They apply for other asset checks as well. |
dd26305
to
d6a4db7
Compare
…b.com/dagster-io/dagster into dpeng817/alert_section_freshness_docs
@dpeng817 Back to you - lmk what you think |
docs/content/concepts/assets/asset-checks/checking-for-data-freshness.mdx
Show resolved
Hide resolved
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.
@erinkcochran87 a few small comments otherwise LGTM. Thanks for putting this together.
Just waiting for BK, then I'll merge this guy |
Adds an alerts section to the freshness guide. I originally wanted to make more sweeping changes here, but it's proving more difficult than I thought. I think the biggest outstanding problem with the guide is thelack of reference to alerting (main important thing about freshness, so let's fix that first). --------- Co-authored-by: Erin Cochran <erin.k.cochran@gmail.com>
Adds an alerts section to the freshness guide.
I originally wanted to make more sweeping changes here, but it's proving more difficult than I thought. I think the biggest outstanding problem with the guide is thelack of reference to alerting (main important thing about freshness, so let's fix that first).