joaopamaral commented on PR #47701: URL: https://github.com/apache/airflow/pull/47701#issuecomment-2722601083
> We are dropping SLA in Airflow 3 - it will be replaced by a different feature https://cwiki.apache.org/confluence/pages/viewpage.action?pageId=323488182 - I doubt we want to invest any time and effort into improving and optimizing the old feature. Thanks, @potiuk! I saw it, but we’ve been running into some issues with DAG processing time lately, and this change was straightforward to implement. We’re already using it and seeing good results with large DAGs. I was considering adding some unit tests, but since the existing ones are passing — and it’s tricky to define a proper performance test (maybe just tracking how often a method is called) I'll keep as is. -- 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]
