uranusjr opened a new pull request #15397: URL: https://github.com/apache/airflow/pull/15397
This creates a skeleton for the AIP-39 implementation to build on, and refactor the logic handling `@once` schedules out of the DAG class. The current implementation contains a lot of duplicated code, and the PR probably shouldn’t be merged until I can pick most of `next_dagrun_after_date` apart (and ultimately eliminate it entirely and rewrite all the tests against `DAG.next_dagrun_info()` or the various TimeTable classes instead). I want to post this WIP since the trivial `@once` implementation already exposes things missing from the interface outlined in AIP-39, and I had to invent an argument (`between`) to cover it. I want to know how ya’ll think of it 🙂 (Note: The current `TimeTable.next_dagrun_info()` interface is missing the session argument from AIP-39 because I don’t need it yet. It will be added when it need to be.) --- **^ Add meaningful description above** Read the **[Pull Request Guidelines](https://github.com/apache/airflow/blob/master/CONTRIBUTING.rst#pull-request-guidelines)** for more information. In case of fundamental code change, Airflow Improvement Proposal ([AIP](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvements+Proposals)) is needed. In case of a new dependency, check compliance with the [ASF 3rd Party License Policy](https://www.apache.org/legal/resolved.html#category-x). In case of backwards incompatible changes please leave a note in [UPDATING.md](https://github.com/apache/airflow/blob/master/UPDATING.md). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: [email protected]
