dominikhei commented on issue #35278: URL: https://github.com/apache/airflow/issues/35278#issuecomment-2987707402
> Glue Databrew was added in [#41848](https://github.com/apache/airflow/pull/41848) > > I'll have a look at the remaining Glue ones. Speaking on this. The `GlueJobCompleteTrigger` currently does not come with a waiter in boto3. @ellisms as you implemented the Operator which pattern do you think is preferable? Is there something I am overlooking. 1. Not letting the Trigger inherit from AwsBaseWaiterTrigger and just port the Sensor. This is like it is done with the [GlueCatalogPartitionSensor](https://github.com/apache/airflow/blob/main/providers/amazon/src/airflow/providers/amazon/aws/sensors/glue_catalog_partition.py#L36) which inherits from the base class, whilst its trigger [does not](https://github.com/apache/airflow/blob/main/providers/amazon/src/airflow/providers/amazon/aws/triggers/glue.py#L78) 2. Or create a new custom waiter and port the Trigger to the `AwsBaseWaiterTrigger`? -- 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]
