From 1f2223927141f11055c740c9c047ba007ab81e14 Mon Sep 17 00:00:00 2001 From: Peter Amstutz Date: Fri, 2 Dec 2022 11:30:43 -0500 Subject: [PATCH] 18842: One more refactor Arvados-DCO-1.1-Signed-off-by: Peter Amstutz --- sdk/python/arvados/keep.py | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/sdk/python/arvados/keep.py b/sdk/python/arvados/keep.py index 8dbad2abb1..b9e22748c1 100644 --- a/sdk/python/arvados/keep.py +++ b/sdk/python/arvados/keep.py @@ -346,15 +346,12 @@ class KeepBlockCache(object): # to free up resources, which would make the operation # succeed. slot.set(blob) - self.cap_cache() except Exception as e: # It failed again. Give up. + slot.set(None) raise arvados.errors.KeepCacheError("Unable to save block %s to disk cache: %s" % (slot.locator, e)) - finally: - # Set the notice that that we are done with the cache - # slot one way or another. - slot.ready.set() + self.cap_cache() class Counter(object): def __init__(self, v=0): -- 2.30.2