demobox commented on this pull request.


> @@ -148,8 +149,13 @@ public boolean blobExists(final String containerName, 
> final String blobName) {
    }
 
    @Override
-   public Iterable<String> getBlobKeysInsideContainer(final String 
containerName) {
-      return containerToBlobs.get(containerName).keySet();
+   public Iterable<String> getBlobKeysInsideContainer(final String 
containerName, String prefix) {
+      ConcurrentSkipListMap<String, Blob> blobs = 
containerToBlobs.get(containerName);
+      if (prefix == null) {
+         return blobs.keySet();
+      }
+      String lastPrefix = prefix + (char) 65535;  // TODO: better sentinel?

[minor] Perhaps create a constant `SENTINEL` or so just to make it clear that 
that's what the last character is supposed to be?

That might be confusing, especially if the TODO is removed in future?

-- 
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1268#discussion_r252093248

Reply via email to