amoghrajesh commented on code in PR #71211:
URL: https://github.com/apache/airflow/pull/71211#discussion_r3772474228
##########
providers/amazon/src/airflow/providers/amazon/aws/operators/glue.py:
##########
@@ -158,6 +203,16 @@ def __init__(
),
**kwargs,
):
+ if resume_glue_job_on_retry is not None:
+ # Kept as a real named parameter (not **kwargs) so `default_args`
still applies correctly.
Review Comment:
Wanted to provide an update on this one. I thought more abnout this and went
a step further than the original fix:
Below Airflow 3.3, durable's value is now fully inert, not just correctly
forwarded via default_args, but ignored outright even if explicitly set, with a
warning.
Below 3.3, only `resume_glue_job_on_retry` controls behavior there, exactly
as it did before this port existed.
Wanted to flag this since it's a further behavior change in the same area
you had already reviewed.
--
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]
