[arvados] updated: 2.1.0-3098-g1f2223927

git repository hosting git at public.arvados.org
Fri Dec 2 16:33:26 UTC 2022


Summary of changes:
 sdk/python/arvados/keep.py | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)

       via  1f2223927141f11055c740c9c047ba007ab81e14 (commit)
      from  ff3645c9c262f5a2aca67ba42ba86084d0e9978a (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 1f2223927141f11055c740c9c047ba007ab81e14
Author: Peter Amstutz <peter.amstutz at curii.com>
Date:   Fri Dec 2 11:30:43 2022 -0500

    18842: One more refactor
    
    Arvados-DCO-1.1-Signed-off-by: Peter Amstutz <peter.amstutz at curii.com>

diff --git a/sdk/python/arvados/keep.py b/sdk/python/arvados/keep.py
index 8dbad2abb..b9e22748c 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):

-----------------------------------------------------------------------


hooks/post-receive
-- 




More information about the arvados-commits mailing list