takayoshi-makabe commented on issue #71246: URL: https://github.com/apache/airflow/issues/71246#issuecomment-5210623584
Tested apache-airflow-providers-openai==1.8.2rc2 for #69506. Installed Airflow 3.3.0 + the RC into a clean venv from PyPI and exercised OpenAITriggerBatchOperator.execute_complete / validate_execute_complete_event / OpenAIBatchTrigger.run() directly (no live OpenAI API needed, this is pure trigger-event branching logic). As a control, I also installed the pre-fix 1.8.1 and confirmed all three bugs actually reproduce there: | Case | 1.8.1 (pre-fix) | 1.8.2rc2 | |---|---|---| | `status="cancelled"` | treated as SUCCESS, returns `batch_id` | raises `OpenAIBatchJobException` | | `event=None` | crashes with raw `TypeError: 'NoneType' object is not subscriptable` | raises `OpenAITriggerEventError` | | unknown `status` (e.g. `"expired"`) | silently treated as SUCCESS | raises `OpenAITriggerEventError` | | `status="success"` / `"error"` | works | still works | Also confirmed OpenAIBatchTrigger.run() now yields exactly one terminal event (no spurious duplicate error event after a terminal event). All checks pass on the RC. No issues found. --- Drafted-by: Claude Code (Sonnet 5); reviewed by @takayoshi-makabe before posting -- 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]
