The GitHub Actions job "Tests (AMD)" on 
airflow.git/fix-go-sdk-heartbeat-nil-check has succeeded.
Run started by GitHub user ColtenOuO (triggered by ColtenOuO).

Head commit for run:
fd40360d35777017ae2b7f9d5c5826ff97406c38 / ColtenOuO <[email protected]>
Go SDK: fix operator-precedence bug that could crash the worker process

heartbeater.Run checked `resp != nil && resp.StatusCode() == 404 ||
resp.StatusCode() == 409` -- `&&` binds tighter than `||`, so the nil
guard only covered the 404 branch. A GeneralHTTPError with a nil
Response (e.g. a transport-level failure) hit resp.StatusCode() on a
nil pointer in the heartbeater's own goroutine, which has no recover,
crashing the whole worker process instead of just failing the task.
Add parentheses to group both status checks under the nil guard.

Report URL: https://github.com/apache/airflow/actions/runs/29764361967

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to