HyukjinKwon opened a new pull request, #57098:
URL: https://github.com/apache/spark/pull/57098

   ### What changes were proposed in this pull request?
   
   **[DO-NOT-MERGE draft — flaky-test deflake, verification in progress]**
   
   Fixes a flaky failure in the `build_java17` scheduled workflow on 
`branch-4.1`.
   `PostgresKrbIntegrationSuite` aborted in `beforeAll` because the Docker 
image pull hit a transient registry error (`HTTP 502 Bad Gateway`). 
`DockerJDBCIntegrationSuite` pulled the image with a single attempt, so one 
transient 502 (or a one-off pull timeout) aborts the entire suite even though 
the rest of the run had zero test failures.
   
   This wraps the image pull in a bounded retry loop 
(`spark.test.docker.imagePullAttempts`, default `3`) with linear backoff. 
Transient failures are retried; the last failure is rethrown once all attempts 
are exhausted.
   
   ### Why are the changes needed?
   
   A single transient registry hiccup should not abort a whole Docker 
integration suite that otherwise passes.
   
   ### Does this PR introduce _any_ user-facing change?
   
   No. Test-infrastructure change only.
   
   ### How was this patch tested?
   
   Existing Docker integration suites exercise this path; verified via fork CI 
(linked below). The change uses only symbols already imported/used in the file 
and stays within scalastyle limits.
   
   ### CI verification
   
   - branch-4.1 (build_java17, docker-integration-tests): _pending_
   


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to