radhwene commented on PR #68361:
URL: https://github.com/apache/airflow/pull/68361#issuecomment-5171665128
Rebased on current `main` today. No conflicts, GitHub reports it as
mergeable.
The changes requested on 2026-06-12 got fixed the next day.
`operation_in_progress_retry()`
used to wrap all of `import_instance`, polling included, so a retryable
error while waiting
re-ran the whole method and submitted the import a second time. The submit
now sits in
`_submit_import`, and only that call carries the decorator. Polling runs
outside the retry
scope, so if it fails the task fails instead of importing the same data
twice. There's a
regression test asserting exactly one submit when polling raises a
retryable error.
The changes-requested review is still open seven weeks later, and it's the
only thing
holding this back. Could someone take another look? I'll rebase again if
it goes stale.
--
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]
