rjgoyln opened a new pull request, #72376:
URL: https://github.com/apache/airflow/pull/72376

   ## Summary
   
   `HttpEventTrigger` wrapped its whole poll loop in one `except Exception` 
that logged `str(e)` and returned. Any error — a 503, a connection reset, a 
broken `response_check_path` — ended the generator, so the asset watcher 
stopped firing even when the next poll would have succeeded, and 
`poll_interval` was skipped because the sleep only sat on the success path. The 
operator was left with one line carrying no exception type and no traceback; 
because the exception was swallowed rather than raised, the triggerer recorded 
none either.
   
   Cancellation is unaffected: `CancelledError` and `GeneratorExit` are 
`BaseException`, and the backoff sleep stays outside the `try`.
   
   ## Tests
   
   `test_trigger_on_post_with_data` relied on the swallowed exception to end 
the generator, so it now drives one successful poll instead.
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Opus 5)
   
   Generated-by: Claude Code (Opus 5) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   


-- 
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]

Reply via email to