[PR] Fix recent change to Batch operator's retry_strategy [airflow]

2023-11-22 Thread via GitHub
ferruzzi opened a new pull request, #35808: URL: https://github.com/apache/airflow/pull/35808 Should fix https://github.com/apache/airflow/pull/35789 "attempts" is a required field is retry_strategy is used. By doing it this way, we can also verify that a user-provided retry_strategy

Re: [PR] Fix recent change to Batch operator's retry_strategy [airflow]

2023-11-22 Thread via GitHub
ferruzzi commented on PR #35808: URL: https://github.com/apache/airflow/pull/35808#issuecomment-1823694970 One possible point to discuss: The value must be between 1 and 10. The API will return a ClientError if a value outside that range is provided. Do we want to catch that at init or

Re: [PR] Fix recent change to Batch operator's retry_strategy [airflow]

2023-11-22 Thread via GitHub
eladkal merged PR #35808: URL: https://github.com/apache/airflow/pull/35808 -- 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.