From e4c24cf0526d3c3e575323d49acdd0f91ff6f7f7 Mon Sep 17 00:00:00 2001 From: Martin Polden Date: Wed, 10 May 2023 11:20:18 +0200 Subject: Always complete non-retriable operations --- client/go/internal/vespa/document/dispatcher.go | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) (limited to 'client') diff --git a/client/go/internal/vespa/document/dispatcher.go b/client/go/internal/vespa/document/dispatcher.go index 00b7dbb411a..7237a87b7e2 100644 --- a/client/go/internal/vespa/document/dispatcher.go +++ b/client/go/internal/vespa/document/dispatcher.go @@ -48,8 +48,6 @@ func (op documentOp) resetResult() documentOp { return op } -func (op documentOp) complete() bool { return op.result.Success() || op.attempts == maxAttempts } - func NewDispatcher(feeder Feeder, throttler Throttler, breaker CircuitBreaker, output io.Writer, verbose bool) *Dispatcher { d := &Dispatcher{ feeder: feeder, @@ -150,7 +148,7 @@ func (d *Dispatcher) processResults() { d.statsMu.Unlock() if d.shouldRetry(op, op.result) { d.enqueue(op.resetResult(), true) - } else if op.complete() { + } else { d.inflightWg.Done() } d.dispatchNext(op.document.Id) -- cgit v1.2.3