dominikhei opened a new pull request, #52243: URL: https://github.com/apache/airflow/pull/52243
--- Part of #35278 I have implemented a custom waiter and ported the sensor and trigger to the base AWS classes. For the naming of parameters and what to include in the docstrings, I looked at the already existing Glue ones. On that note: I noticed that all operators use the `max_attempts` and `poll_interval` parameters (although often named differently), that are later passed to the trigger. Wouldn't it make sense to move them to the `AWSBaseOperator` too? -- 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: commits-unsubscr...@airflow.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
