vincbeck commented on code in PR #27820:
URL: https://github.com/apache/airflow/pull/27820#discussion_r1029507161


##########
airflow/providers/amazon/aws/operators/redshift_cluster.py:
##########
@@ -498,22 +502,38 @@ def __init__(
         wait_for_completion: bool = True,
         aws_conn_id: str = "aws_default",
         poll_interval: float = 30.0,
+        retry: bool = False,
+        retry_attempts: int = 10,

Review Comment:
   That's a good idea. The only downside of it is you dont provide a default 
value in case you want to retry. If you want to retry, you have to come up with 
a number of retries which can be not very intuitive for users



-- 
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]

Reply via email to