Clarify warning for partition-driven DAGs in dags next-execution#61319
Clarify warning for partition-driven DAGs in dags next-execution#61319shleshaG wants to merge 1 commit intoapache:mainfrom
Conversation
|
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)
|
bugraoz93
left a comment
There was a problem hiding this comment.
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
|
Actualy - the right fix here will be to remove interval. In Airflow 3 we have |
|
@potiuk Thanks a lot for the clarification! That makes sense — I see now that updating the warning text alone is not sufficient. |
| "This DAG may have schedule interval '@once' or `None`.", | ||
| "or have a schedule of '@once' or None.", |
There was a problem hiding this comment.
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.
What does this PR do?
Clarifies the warning message shown by
airflow dags next-executionwhen no nextdata 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
@onceorNoneschedules, which could be misleading.This change improves CLI clarity without altering behavior.
How was this tested?
Related issue