ozemin opened a new pull request, #70218: URL: https://github.com/apache/airflow/pull/70218
Adds `deferrable=True` support to `QuickSightCreateIngestionOperator` and `QuickSightSensor`, following the same waiter-based pattern as the rest of the Amazon provider. QuickSight has no official boto3 waiters (boto/boto3#2769), so this adds a custom `ingestion_complete` waiter and a `QuickSightIngestionCompletedTrigger`, and replaces the hook's hand-written polling loop, which held a worker slot with `time.sleep` and had no upper bound. One thing without a direct precedent: `DescribeIngestion` requires `AwsAccountId`, so it is resolved on the worker before deferring and passed to the trigger — the triggerer never makes a sync STS call. Behaviour changes: the synchronous wait is now bounded by `waiter_max_attempts` (default 60 attempts × 30s) instead of waiting forever; noted in the provider changelog. `QuickSightHook.wait_for_state` and the operator's `check_interval` are deprecated (still working) in favour of `wait_for_ingestion` / `waiter_delay`. Failures raise `QuickSightIngestionFailedError`, an `AirflowException` subclass. --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code -- 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]
