andreahlert commented on PR #64590: URL: https://github.com/apache/airflow/pull/64590#issuecomment-4215016258
> Actually - the assumption here is wrong - the data is cached mostly - with 4 hr TTL already and stored in ~/.build directory. Or there is something wrong with it and we need to fix it? Maybe the PR description wasn't clear enough on this, I'll update it. Imho, No overlap actually. status_cache stores branch-level stuff (recent failures on main, canary builds) to detect systemic CI issues. Different key space. What I'm caching here is per-commit: check counts keyed by SHA with no TTL since completed checks never change for the same commit, saves the paginated GraphQL query on re-runs. Workflow runs with a 10min TTL (up to 3 REST calls per PR). Author profiles with 7d TTL since they barely change between sessions. Also added generate_review_questions which extracts deterministic checks from the local diff (large PR warnings, missing tests, version field validation) and feeds them into the LLM prompt without any extra API call. -- 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]
