gopidesupavan commented on issue #49648: URL: https://github.com/apache/airflow/issues/49648#issuecomment-2887166706
> [@gopidesupavan](https://github.com/gopidesupavan) Sorry I got to check this issue today. I am missing some context here. I see that the related dag issue linked here is related to "max_consecutive_failed_dag_runs" which is set as default to -1 in TaskSDK and is fixed now to 0( Same count as what was in serialized dagbag). How is this issue related to it? Could you give me more context on that part? Thank you. Yes @Bowrna this issue not directly related to that we have fixed, but its a side effect we have noticed, In the `round trip example test` we parse the DAG and compares the serialised fields. so at present there is inconsistency some examples uses old way importing DAG definition and some or new way. we would like all the DAG imports from task sdk. thats the ideal way and users should be using, so when users sees our example and found old imports they may confuse. :) . I will take a look your changes tomorrow. Thanks for taking this up. -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
