psg2 commented on pull request #16842:
URL: https://github.com/apache/airflow/pull/16842#issuecomment-875968087
Got it, I guess it was a miscommunication issue then, I asked on Slack about
this when I noticed that `/dags/{dag_id}` returned a DAG that was inactive
while the `/dag/{dag_id}/details` fails in such case that there is no more the
associated dag on the SerializedDag table and I thought they should have the
same behavior. Maybe I didn't express well my thoughts on Slack or
misunderstood the response.
Slack Thread:
https://apache-airflow.slack.com/archives/CCPRP7943/p1625504228302500
> I don't understand why you want to make this change? What do you mean by
"stale DAGs"?
I was thinking stale DAGs were inactive DAGs that no longer have the file on
the filesystem at the time.
Now after reading the `/dags/{dag_id}` description which is `Presents only
information available in database (DAGModel). If you need detailed information,
consider using GET /dags/{dag_id}/details.` it actually looks correct as it is
today.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
