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.

Sign up for GitHub

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

Jump to bottom

Refine data depends_on dependency checks #29682

Merged
merged 2 commits into from
Oct 4, 2021
Merged

Refine data depends_on dependency checks #29682

merged 2 commits into from
Oct 4, 2021

Commits on Sep 30, 2021

  1. don't check all ancestors for data depends_on 

    Only depends_on ancestors for transitive dependencies when we're not
pointed directly at a resource. We can't be much more precise here,
since in order to maintain our guarantee that data sources will wait for
explicit dependencies, if those dependencies happen to be a module,
output, or variable, we have to find some upstream managed resource in
order to check for a planned change.
    @jbardin
    jbardin committed Sep 30, 2021
    Configuration menu
    Copy the full SHA
    016463e View commit details
    Browse the repository at this point in the history

  2. test for unexpected data reads

    @jbardin
    jbardin committed Sep 30, 2021
    Configuration menu
    Copy the full SHA
    618e9cf View commit details
    Browse the repository at this point in the history