20318: Fix BadChecksum error not propagated through cache layer.

[arvados.git] / sdk / go / arvados / keep_cache.go

diff --git a/sdk/go/arvados/keep_cache.go b/sdk/go/arvados/keep_cache.go
index 6cf2e23d7e4dfa868feca01cc27a61a5cf06e241..a5444652721921b016b4c2c85c95df8aec6e80ad 100644 (file)
--- a/sdk/go/arvados/keep_cache.go
+++ b/sdk/go/arvados/keep_cache.go
@@ -358,11 +358,10 @@ func (cache *DiskCache) ReadAt(locator string, dst []byte, offset int) (int, err
        for !progress.done && progress.size < len(dst)+offset {
                progress.cond.Wait()
        }
-       ok := progress.size >= len(dst)+offset
        err = progress.err
        progress.cond.L.Unlock()
 
-       if !ok && err != nil {
+       if err != nil {
                // If the copy-from-backend goroutine encountered an
                // error before copying enough bytes to satisfy our
                // request, we return that error.