Conversation
Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
|
👋 @haritamar |
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
Disabled knowledge base sources:
📝 WalkthroughWalkthroughThe warehouse test workflow timeout increases from 60 to 90 minutes. ChangesWarehouse test timeout
Estimated code review effort: 1 (Trivial) | ~2 minutes ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Summary
test-warehouse.ymltimeout-minutes: 60 -> 90.The
latest_official, databricks_catalogjob already runs 47–60 min on recent branches (a pass at 59m46s on #1050's earlier run), and #1050 adds 19test_metric_stabilitytests at ~1–4 min each on Databricks, which pushed it to 99% complete when cancelled in https://github.com/elementary-data/dbt-data-reliability/actions/runs/34201868801. The Spark job in the same run also hit the limit, but that was a 50‑minute hang in the Spark Docker build'swgetdownloads (infra flake), not test runtime.Link to Devin session: https://app.devin.ai/sessions/d520e797006a405d96e2fed89117a356
Open in Devin Desktop: https://app.devin.ai/desktop/session/d520e797006a405d96e2fed89117a356?variant=devin
Requested by: @haritamar
Summary by CodeRabbit