The GitHub Actions job "Tests" on airflow.git has failed. Run started by GitHub user potiuk (triggered by potiuk).
Head commit for run: 6b4aff89343cbd613366f3e963f25069370cfae9 / Ash Berlin-Taylor <[email protected]> Remove caching inside ExecutorLoader in favour of `job.executors` property (#44367) The primary drivers of this are simplification, specifically: 1. It avoids global state -- which makes things easier to reason about. For instance this removes the use of `reload()` in our tests; a sure sign that things are overly complicated 2. It removes the wierd "mixed" source of truth inside SchedulerJob, where some places were still looking at `self.job.executors` while others were looking at ExecutorLoader.load_executor directly `set_default_executor` was removed as nothing called it. The comment about "Cache this executor by name here, so we can look it up later if it is requested again, and not have to construct a new object" is moot as the SchedulerJob is the only thing that has persistent executor objects, and `self.job.executors` is already that cache. I noticed a small glitch with test_retry_handling_job and upon examination discovered that it wasn't useful anymore so removed the test and the dag file as nothing else uses it. Report URL: https://github.com/apache/airflow/actions/runs/12031755504 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
