From 2a6c82d3e9a20e0b5ac2267f45e89f8d009b0f2c Mon Sep 17 00:00:00 2001 From: Tom Clegg Date: Fri, 10 Mar 2023 13:35:09 -0500 Subject: [PATCH] 19972: Remove unused code. Arvados-DCO-1.1-Signed-off-by: Tom Clegg --- sdk/go/arvados/client.go | 34 ---------------------------------- 1 file changed, 34 deletions(-) diff --git a/sdk/go/arvados/client.go b/sdk/go/arvados/client.go index 1ccf187ea6..d1a87e30e8 100644 --- a/sdk/go/arvados/client.go +++ b/sdk/go/arvados/client.go @@ -230,44 +230,10 @@ func NewClientFromEnv() *Client { } } -func shouldRetry(req *http.Request, resp *http.Response, err error) bool { - if nerr := net.Error(nil); errors.As(err, &nerr) && nerr.Temporary() { - return true - } - switch req.Method { - case "GET", "HEAD", "PUT", "OPTIONS", "DELETE": - default: - return false - } - if uerr := new(url.Error); errors.As(err, &uerr) && uerr.Err.Error() == "Service Unavailable" { - // This is how http.Client reports 503 from proxy server - return true - } - if err != nil { - return false - } - switch resp.StatusCode { - case 408, 409, 422, 423, 500, 502, 503, 504: - return true - default: - return false - } -} - var reqIDGen = httpserver.IDGenerator{Prefix: "req-"} var nopCancelFunc context.CancelFunc = func() {} -func (c *Client) checkRetry(ctx context.Context, resp *http.Response, err error) (bool, error) { - if c.requestLimiter.Report(resp, err) { - c.last503.Store(time.Now()) - } - if c.Timeout == 0 { - return false, err - } - return retryablehttp.DefaultRetryPolicy(ctx, resp, err) -} - // Do augments (*http.Client)Do(): adds Authorization and X-Request-Id // headers, delays in order to comply with rate-limiting restrictions, // and retries failed requests when appropriate. -- 2.39.5