diff options
author | Bjørn Christian Seime <bjorn.christian@seime.no> | 2023-08-23 10:28:46 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-08-23 10:28:46 +0200 |
commit | d4ffa9cc1f32fa2024e13f5ee7f1d515f071aa87 (patch) | |
tree | ef116969ce9f5d3ea309d97ddaada999e69f09b1 /client | |
parent | 3c6569ea018230aa785717f78a4c0812245ce511 (diff) | |
parent | 024d35283f91b1bc22e67eeb3be1c9cb334eb320 (diff) |
Merge pull request #28118 from vespa-engine/jonmv/feed-503-is-not-just-backpressure
503 from /doc/v1 is not backpressure, but hopefully transient
Diffstat (limited to 'client')
-rw-r--r-- | client/go/internal/vespa/document/dispatcher.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/client/go/internal/vespa/document/dispatcher.go b/client/go/internal/vespa/document/dispatcher.go index fb7a532e332..b76b6b44016 100644 --- a/client/go/internal/vespa/document/dispatcher.go +++ b/client/go/internal/vespa/document/dispatcher.go @@ -110,10 +110,10 @@ func (d *Dispatcher) shouldRetry(op documentOp, result Result) bool { d.throttler.Success() d.circuitBreaker.Success() return false - } else if result.HTTPStatus == 429 || result.HTTPStatus == 503 { + } else if result.HTTPStatus == 429 { d.throttler.Throttled(d.inflightCount.Load()) return true - } else if result.Err != nil || result.HTTPStatus == 500 || result.HTTPStatus == 502 || result.HTTPStatus == 504 { + } else if result.Err != nil || result.HTTPStatus == 500 || result.HTTPStatus == 502 || result.HTTPStatus == 503 || result.HTTPStatus == 504 { d.circuitBreaker.Failure() if retry { return true |