aglinxinyuan commented on PR #7623: URL: https://github.com/apache/texera/pull/7623#issuecomment-5300975642
All four addressed in fb5ec5a, plus a description edit. - **`Closes #6073`** — promoted from "Related to". The two follow-ups it lists are exactly what this ships. It's assigned to @rbelavadi for the Windows-side tracking, so say the word if you'd rather it stay open and I'll revert to a plain reference. - **Browser `hookTimeout`** — dropped. Confirmed in the pinned 4.1.10 that `browser.enabled` already resolves 15s/30s, so that line set the value it inherited; the remaining `testTimeout` is now documented as a 15s → 30s bump rather than 6x. - **`timeout-minutes: 30`** on the frontend job, so the "the job's own timeout bounds a true hang" sentence is backed by a real key rather than GitHub's implicit 6h cap. Green legs run 4-12 min and the install step alone is allowed 20. - **fail-fast rationale** — now names `platform`, `platform-integration`, `agent-service` and `infra` instead of claiming every multi-leg matrix. `amber-integration` (2 OS legs) and `pyamber` (3 Python versions) are the counterexamples; left alone here, happy to open the follow-up. Description updated to match: per-config timeout rows, the new job cap, and the ceiling split (6x under jsdom, 2x in browser mode). -- 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]
