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

Raise import error if a task uses a non-existent pool #21829

Closed
wants to merge 4 commits into from
Closed

Raise import error if a task uses a non-existent pool #21829

wants to merge 4 commits into from

Commits on Feb 28, 2022

  1. Raise import error if a task uses a non-existent pool 

    This PR proposes to raise import error if a task in a DAG is using
a non-existent pool. This will free the scheduler from continuously
trying to schedule a task with a non-existing pool

fixup! Raise import error if a task uses a non-existent pool

fixup! Raise import error if a task uses a non-existent pool
    @ephraimbuddy
    ephraimbuddy committed Feb 28, 2022
    Configuration menu
    Copy the full SHA
    f752d3c View commit details
    Browse the repository at this point in the history

  2. Update airflow/models/dag.py 

    Co-authored-by: Tzu-ping Chung <uranusjr@gmail.com>
    @ephraimbuddy @uranusjr
    ephraimbuddy and uranusjr committed Feb 28, 2022
    Configuration menu
    Copy the full SHA
    7ce123b View commit details
    Browse the repository at this point in the history

  3. fixup! Update airflow/models/dag.py

    @ephraimbuddy
    ephraimbuddy committed Feb 28, 2022
    Configuration menu
    Copy the full SHA
    ec3fea6 View commit details
    Browse the repository at this point in the history

Commits on Mar 2, 2022

  1. Update airflow/models/dag.py 

    Co-authored-by: Daniel Standish <15932138+dstandish@users.noreply.github.com>
    @ephraimbuddy @dstandish
    ephraimbuddy and dstandish authored Mar 2, 2022
    Configuration menu
    Copy the full SHA
    3457a2e View commit details
    Browse the repository at this point in the history