[ARVADOS] updated: ad69b48e324c3ce29a4d2c84732dfd3d0288ebb3

git at public.curoverse.com git at public.curoverse.com
Mon Feb 23 10:16:47 EST 2015


Summary of changes:
 doc/index.html.liquid            |  2 +-
 sdk/python/arvados/collection.py | 38 ++++++++++++++++++++------------------
 2 files changed, 21 insertions(+), 19 deletions(-)

       via  ad69b48e324c3ce29a4d2c84732dfd3d0288ebb3 (commit)
       via  2548543949ac90b51d555607b115d0e0efd45640 (commit)
       via  d8856736576e492edf08eba46b523880569e5123 (commit)
      from  3070d89e1e52cbc42a04560f5db2630c0c4cbd6b (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 ad69b48e324c3ce29a4d2c84732dfd3d0288ebb3
Merge: 2548543 d885673
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Mon Feb 23 10:16:38 2015 -0500

    Merge branch 'master' into 4823-python-sdk-writable-collection-api


commit 2548543949ac90b51d555607b115d0e0efd45640
Author: Peter Amstutz <peter.amstutz at curoverse.com>
Date:   Mon Feb 23 10:16:08 2015 -0500

    4823: Clean up imports in collection.py

diff --git a/sdk/python/arvados/collection.py b/sdk/python/arvados/collection.py
index 6d5dd4f..a77f309 100644
--- a/sdk/python/arvados/collection.py
+++ b/sdk/python/arvados/collection.py
@@ -3,14 +3,17 @@ import logging
 import os
 import re
 import errno
+import hashlib
 import time
+import threading
 
 from collections import deque
 from stat import *
 
 from .arvfile import split, _FileLikeObjectBase, ArvadosFile, ArvadosFileWriter, ArvadosFileReader, _BlockManager, synchronized, must_be_writable, SYNC_READONLY, SYNC_EXPLICIT, NoopLock
-from keep import *
-from .stream import StreamReader, normalize_stream
+from keep import KeepLocator, KeepClient
+from .stream import StreamReader
+from ._normalize_stream import normalize_stream
 from ._ranges import Range, LocatorAndRange
 from .safeapi import ThreadSafeApiCache
 import config
@@ -786,22 +789,21 @@ class SynchronizedCollectionBase(CollectionBase):
 
         target_dir = self.find_or_create("/".join(targetcomponents[0:-1]), COLLECTION)
 
-        with target_dir.lock:
-            if target_name in target_dir:
-                if isinstance(target_dir[target_name], SynchronizedCollectionBase) and sourcecomponents:
-                    target_dir = target_dir[target_name]
-                    target_name = sourcecomponents[-1]
-                elif not overwrite:
-                    raise IOError((errno.EEXIST, "File already exists"))
-
-            modified_from = None
-            if target_name in target_dir:
-                modified_from = target_dir[target_name]
-
-            # Actually make the copy.
-            dup = source_obj.clone(target_dir)
-            target_dir._items[target_name] = dup
-            target_dir._modified = True
+        if target_name in target_dir:
+            if isinstance(target_dir[target_name], SynchronizedCollectionBase) and sourcecomponents:
+                target_dir = target_dir[target_name]
+                target_name = sourcecomponents[-1]
+            elif not overwrite:
+                raise IOError((errno.EEXIST, "File already exists"))
+
+        modified_from = None
+        if target_name in target_dir:
+            modified_from = target_dir[target_name]
+
+        # Actually make the copy.
+        dup = source_obj.clone(target_dir)
+        target_dir._items[target_name] = dup
+        target_dir._modified = True
 
         if modified_from:
             self.notify(MOD, target_dir, target_name, (modified_from, dup))

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


hooks/post-receive
-- 




More information about the arvados-commits mailing list