Skip to content

Revert "Add resume_glue_job_on_retry to GlueJobOperator (#59392)"#62730

Merged
potiuk merged 1 commit into
mainfrom
revert-resume-glue-job
Mar 3, 2026
Merged

Revert "Add resume_glue_job_on_retry to GlueJobOperator (#59392)"#62730
potiuk merged 1 commit into
mainfrom
revert-resume-glue-job

Conversation

@potiuk

@potiuk potiuk commented Mar 2, 2026

Copy link
Copy Markdown
Member

This reverts commit 8396957.

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)
  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@potiuk potiuk requested a review from o-nikolas as a code owner March 2, 2026 20:24
@boring-cyborg boring-cyborg Bot added area:providers provider:amazon AWS/Amazon - related issues labels Mar 2, 2026
@o-nikolas

o-nikolas commented Mar 2, 2026

Copy link
Copy Markdown
Contributor

Relates to:

@potiuk

potiuk commented Mar 2, 2026

Copy link
Copy Markdown
Member Author

cc: @henry3260 @wilsonhooi86 -> since the testing is not yet completed for #62560 I most likely revert #59392 -> unless you confirm the fix works, otherwise @henry3260 you will have to redo/test both.

@potiuk

potiuk commented Mar 2, 2026

Copy link
Copy Markdown
Member Author

I wil decide in about an hour or so when I release current wave.

@potiuk potiuk merged commit 26f85cf into main Mar 3, 2026
90 checks passed
@potiuk potiuk deleted the revert-resume-glue-job branch March 3, 2026 00:19
@potiuk

potiuk commented Mar 3, 2026

Copy link
Copy Markdown
Member Author

@henry3260 @wilsonhooi86 - you will have to redo this :(

@wilsonhooi86

wilsonhooi86 commented Mar 3, 2026 •
edited
Loading

Copy link
Copy Markdown

Hi @potiuk , thanks for letting us know. Sure, I believe we can redo for next release.

Hi @henry3260 , thank you so much for your effort and your effort is not a waste. I'm sorry I not able to test it on time. Let's try to work on next release if that's ok with you. Perhaps can continue the changes in this existing MR: #62560 for next release? Let me know your thoughts =)

@henry3260

henry3260 commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

Hi @potiuk , thanks for letting us know. Sure, I believe we can redo for next release.

Hi @henry3260 , thank you so much for your effort and your effort is not a waste. I'm sorry I not able to test it on time. Let's try to work on next release if that's ok with you. Perhaps can continue the changes in this existing MR: #62560 for next release? Let me know your thoughts =)

Sounds good! Let's aim for the next release. ：）

dominikhei pushed a commit to dominikhei/airflow that referenced this pull request Mar 11, 2026
@potiuk @dominikhei
Revert "Add resume_glue_job_on_retry to GlueJobOperator (apache#59392)…
c7583e5 
…" (apache#62730)

This reverts commit 8396957.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@eladkal eladkal eladkal approved these changes

@o-nikolas o-nikolas o-nikolas approved these changes

@vincbeck vincbeck vincbeck approved these changes

Assignees

No one assigned

Labels

area:providers provider:amazon AWS/Amazon - related issues

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@potiuk @o-nikolas @wilsonhooi86 @henry3260 @eladkal @vincbeck