Skip to content

Clarify warning for partition-driven DAGs in dags next-execution#61319

Open
shleshaG wants to merge 1 commit intoapache:mainfrom
shleshaG:fix-dag-next-execution
Open

Clarify warning for partition-driven DAGs in dags next-execution#61319
shleshaG wants to merge 1 commit intoapache:mainfrom
shleshaG:fix-dag-next-execution

Conversation

@shleshaG
Copy link

@shleshaG shleshaG commented Feb 1, 2026

What does this PR do?

Clarifies the warning message shown by airflow dags next-execution when no next
data interval can be determined, noting that this can also occur for
dataset- or partition-driven DAGs.

Why is this needed?

With AIP-76, partition-driven and dataset-driven DAGs may not have a
determinable next execution date. The previous message only mentioned
@once or None schedules, which could be misleading.

This change improves CLI clarity without altering behavior.

How was this tested?

  • Manual verification of CLI output

Related issue

@boring-cyborg
Copy link

boring-cyborg bot commented Feb 1, 2026

Congratulations on your first Pull Request and welcome to the Apache Airflow community! If you have any issues or are unsure about any anything please check our Contributors' Guide (https://github.com/apache/airflow/blob/main/contributing-docs/README.rst)
Here are some useful points:

  • Pay attention to the quality of your code (ruff, mypy and type annotations). Our prek-hooks will help you with that.
  • In case of a new feature add useful documentation (in docstrings or in docs/ directory). Adding a new operator? Check this short guide Consider adding an example DAG that shows how users should use it.
  • Consider using Breeze environment for testing locally, it's a heavy docker but it ships with a working Airflow and a lot of integrations.
  • Be patient and persistent. It might take some time to get a review or get the final approval from Committers.
  • Please follow ASF Code of Conduct for all communication including (but not limited to) comments on Pull Requests, Mailing list and Slack.
  • Be sure to read the Airflow Coding style.
  • Always keep your Pull Requests rebased, otherwise your build might fail due to changes not related to your commits.
    Apache Airflow is a community-driven project and together we are making it better 🚀.
    In case of doubts contact the developers at:
    Mailing List: dev@airflow.apache.org
    Slack: https://s.apache.org/airflow-slack

Copy link
Contributor

@bugraoz93 bugraoz93 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't quite get the change here as explanation looks the same but I am expecting some kind of implementation for the issue. Maybe you can try to ask more questions there to understand exact change needs to be made

@potiuk
Copy link
Member

potiuk commented Feb 3, 2026

Actualy - the right fix here will be to remove interval. In Airflow 3 we have schedule as the preferred way to specify schedule - other past ways are deprecated, so just removing interval will be enough - adding almost the same thing twice is quite confusing

@shleshaG
Copy link
Author

shleshaG commented Feb 3, 2026

@potiuk Thanks a lot for the clarification! That makes sense — I see now that updating the warning text alone is not sufficient.
Based on your guidance, I’ll update the implementation to remove usage of interval and rely solely on schedule, which aligns with the Airflow 3 scheduling model. I’ll push an updated commit shortly. Please let me know if there’s anything else I should watch out for.

Comment on lines 338 to +339
"This DAG may have schedule interval '@once' or `None`.",
"or have a schedule of '@once' or None.",
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are actually more cases, once and none are simply the oldest ones (and probably the only ones when this message was first introduced).

Let’s change the entire message to something like This Dag may have a schedule that is not time-based, such as None or @once.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants