reptillicus commented on this pull request.


> +            // should also get deleted
+            delete(object);
+         }
+         else if (object.isDirectory() & (optsPrefix.endsWith(File.separator) 
| isNullOrEmpty(optsPrefix))) {
+            // S3 blobstores will only match prefixes that end with a trailing 
slash/file separator
+            // For insance, if we have a blob at /path/1/2/a, a prefix of 
/path/1/2 will not list /path/1/2/a
+            // but a prefix of /path/1/2/ will
+            File containerFile = openFolder(container + File.separator + 
normalizedOptsPath);
+            File[] children = containerFile.listFiles();
+            if (null != children) {
+               for (File child : children) {
+                  if (options.isRecursive()) {
+                     Utils.deleteRecursively(child);
+                  } else {
+                     if (child.isFile()) {
+                        Utils.delete(child);

It might be possible, but `removeDirectoriesTreeOfBlobKey` walks up the tree, 
and this one is walking deeper into the tree. Suggestions? 

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1258#discussion_r245168313

Reply via email to