Skip to content

Commit 09d73a0

Browse files
authored
GH-44449: [Release] Retry on HTTP error in binary upload (#44450)
### Rationale for this change Artifactory sometimes reports the following 503 HTTP error: ```text BinaryTask::ArtifactoryClient::Error: failed to request: https://apache.jfrog.io/artifactory/arrow/debian-rc/pool/bookworm/main/a/apache-arrow/libparquet-glib1800_18.0.0-1_arm64.deb: PUT: 503 { "errors" : [ { "status" : 503, "message" : "503 : Failed during addStream to bucket" } ] } ``` It seems that this is a temporary error. So we can retry on this type error for stable upload. ### What changes are included in this PR? Retry on HTTP error. ### Are these changes tested? Yes. ### Are there any user-facing changes? No. * GitHub Issue: #44449 Authored-by: Sutou Kouhei <kou@clear-code.com> Signed-off-by: Sutou Kouhei <kou@clear-code.com>
1 parent 88e6c8f commit 09d73a0

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

dev/release/binary-task.rb

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -532,7 +532,8 @@ def with_retry(max_n_retries, target)
532532
OpenSSL::OpenSSLError,
533533
SocketError,
534534
SystemCallError,
535-
Timeout::Error => error
535+
Timeout::Error,
536+
Error => error
536537
n_retries += 1
537538
if n_retries <= max_n_retries
538539
$stderr.puts

0 commit comments

Comments
 (0)