jens-scheffler-bosch commented on PR #35243: URL: https://github.com/apache/airflow/pull/35243#issuecomment-1783937825
Yes, in deep I was also scratching my head. Obviously there is a kind of basic caching but also with expiry check. The main driver for the `lru_cache` was the use in `_get_next_dagruns_to_examine` for the case if 200+ times the same DAG is queued. Then we don't need to check during one iteration over-and-over whether the DAG changed in between. That was the intend. But the in-deep-investigation - and therefore the previous PR was made by @AutomationDev85 - who is probably back online on Monday. -- 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]
