Cloud-Architect-Emma commented on issue #67213:
URL: https://github.com/apache/airflow/issues/67213#issuecomment-4497142936
Verified apache-airflow-providers-amazon==9.29.0rc1 (#65571)
I verified that the `SageMakerNotebookJobTrigger` async implementation
is present and correct in the RC wheel:
- `SageMakerNotebookJobTrigger.run()` is now a proper async generator
using `asyncio.to_thread` for CI-safe async execution
- `serialize()` correctly round-trips all four fields: `execution_id`,
`execution_name`, `waiter_delay`, `waiter_max_attempts`
- Terminal success states (`COMPLETED`, `SUCCEEDED`) and failure states
(`FAILED`, `ERROR`, `CANCELLED`, `STOPPED`) are handled correctly
- Timeout behavior yields a `TriggerEvent` with `status: "failed"` and
`error: "Execution timed out"`
The implementation and all unit tests are present in the RC package.
--
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]
