jaklan commented on PR #43453: URL: https://github.com/apache/airflow/pull/43453#issuecomment-2466681024
> Nothing wrong with incremental changes. The dbt folks were, at least peripherally, involved in the initial creation of this provider. Continued collaboration is definitely welcomed. This provider has come a long way since it was created with contributions from folks like you @krzysztof-kubis, so thank you! As you wish, just to note it's already a breaking change - with the second approach we could avoid it by either: - introducing a new parameter `rerun_from_failure` and deprecating `retry_from_failure` (to differentiate dbt Cloud `rerun` - as that's the API endpoint - from Airflow task `retry`) or - keeping `retry_from_failure`, but supporting all options like: `True | False | "never" | "always" | "when_task_retried"` and a) marking `True | False` as deprecated b) keeping the mapping `True -> "always"`, `False -> "never"` -- 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]
