The GitHub Actions job "Tests" on airflow.git/feat/vault-data-layer has failed. Run started by GitHub user andreahlert (triggered by andreahlert).
Head commit for run: 6709a3ecdea596dd934439ddeb1f9d707ca63213 / André Ahlert <[email protected]> Address code review: atomic writes, partial check guard, thread safety, false positives - Use atomic file writes (temp file + os.replace) in CacheStore.save to prevent corrupt reads from concurrent threads. - Skip caching check status when IN_PROGRESS/QUEUED/PENDING counts are present to avoid persisting incomplete CI results. - Add threading.Lock to _author_profile_cache to prevent redundant API calls from concurrent workers. - Scan only added lines (not removed) in generate_review_questions to avoid false positives from removed deprecation notices. - Document that review questions are active in sequential mode only (diff_text not yet available at background LLM submission time). - Document the intentional use of time.time() for persistent TTLs. - Add tests for scan_cached_pr_numbers and invalidate_stale_caches covering success, corrupt files, stale SHA, and multi-cache scenarios. Signed-off-by: André Ahlert <[email protected]> Report URL: https://github.com/apache/airflow/actions/runs/24161233272 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
