ephraimbuddy commented on pull request #16842:
URL: https://github.com/apache/airflow/pull/16842#issuecomment-876123194
> 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.
Good
--
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]
