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

Refactor DAG.dataset_triggers into the timetable class #39321

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Refactor DAG.dataset_triggers into the timetable class #39321

wants to merge 1 commit into from

Commits on Apr 30, 2024

  1. Move dataset information into timetable 

    This allows us to remove a bunch of conditionals regarding whether a DAG
is backed by a timetable or dataset condition, and a weird edge case in
serialization where we don't actually deserialize datasets in a
timetable.

Now datasets are always serialized as a part of the timetable, and we
always evaluate the timetable and datasets. Timetables that do not
actually contain datasets (most of them) simply always evaluate to
False.
    @uranusjr
    uranusjr committed Apr 30, 2024
    Configuration menu
    Copy the full SHA
    54fdeb7 View commit details
    Browse the repository at this point in the history