1fanwang opened a new pull request, #72420:
URL: https://github.com/apache/airflow/pull/72420

   A synchronous Cloud Batch task uses a caller-stable job ID. If its worker 
exits after submission, retry resubmits the same ID and fails with `409 ... 
already exists`, even if the first job is active or succeeded.
   
   On Airflow 3.3+, the operator saves the canonical name before waiting. Retry 
fetches only that task-owned job: wait if active, restore success, resubmit 
after `NotFound`, or reject other terminal states. Fresh and deferrable 
execution, `durable=False`, and older Airflow behavior do not change.
   
   ## Testing
   
   The regression test drives the real task state accessor and `execute()` with 
a local hook stand-in.
   
   <details>
   <summary>Raw retry proof</summary>
   
   ```console
   $ AIRFLOW_HOME=.build/cloud-batch-proof-home uv run --project 
providers/google pytest 
providers/google/tests/unit/google/cloud/operators/test_cloud_batch.py::TestCloudBatchSubmitJobOperator::test_retry_reconnects_to_running_job
 -xvs
   
   # upstream/main
   E   google.api_core.exceptions.AlreadyExists: 409 Job 
projects/testproject/locations/us-central1/jobs/test already exists
   =================== 1 failed, 1 warning in 113.59s (0:01:53) 
===================
   
   # this branch
   PASSED
   =================== 1 passed, 1 warning in 92.69s (0:01:32) 
====================
   ```
   
   </details>
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes (GitHub Copilot CLI, GPT-5.6 Sol)
   
   Generated-by: GitHub Copilot CLI (GPT-5.6 Sol) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   
   ---
   
   * Read the **[Pull Request 
Guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#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](https://cwiki.apache.org/confluence/display/AIRFLOW/Airflow+Improvement+Proposals))
 is needed.
   * When adding dependency, check compliance with the [ASF 3rd Party License 
Policy](https://www.apache.org/legal/resolved.html#category-x).
   * For significant user-facing changes create newsfragment: 
`{pr_number}.significant.rst`, in 
[airflow-core/newsfragments](https://github.com/apache/airflow/tree/main/airflow-core/newsfragments).
 You can add this file in a follow-up commit after the PR is created so you 
know the PR number.
   


-- 
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]

Reply via email to