andygrove commented on PR #5489: URL: https://github.com/apache/datafusion-comet/pull/5489#issuecomment-5721101241
> Triage note: #5881 is doing the same thing — bootstrapping Maven from the setup actions so every `mvnw` job survives a 403 or 429 from `repo.maven.apache.org`. It touches four of the same files: `.github/actions/java-test/action.yaml`, `ci.yml`, `pr_build_linux.yml` and `dev/ci/compute-changes.py`. > > That one is mine and it is the newer of the two, so I plainly missed this when I opened it. This PR also retries the artifact downloads rather than only the distribution, which #5881 does not do at all. Would you rather take the combined change here, or shall I rebase #5881 onto whatever survives? @sunchao I went ahead and merged https://github.com/apache/datafusion-comet/pull/5881 since it was green in CI and already approved. Maybe you could rebase this one after that merges? -- 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]
