Re-optimize LocalBlobStore.getBlob with ranges

This fixes a memory regression from 8de7b69 where the transient BlobStore changed from a ByteSource to a byte[].
apache · Jun 29, 2021 · d1bd51f · d1bd51f
1 parent 2791f47
commit d1bd51f
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/blobstore/src/main/java/org/jclouds/blobstore/config/LocalBlobStore.java b/blobstore/src/main/java/org/jclouds/blobstore/config/LocalBlobStore.java
@@ -682,10 +682,13 @@ public Blob getBlob(String containerName, String key, GetOptions options) {
 
             // Try to convert payload to ByteSource, otherwise wrap it.
             ByteSource byteSource;
-            try {
-               byteSource = (ByteSource) blob.getPayload().getRawContent();
-            } catch (ClassCastException cce) {
-               // This should not happen; both FilesystemStorageStrategyImpl and TransientStorageStrategy return ByteSource
+            Object object = blob.getPayload().getRawContent();
+            if (object instanceof ByteSource) {
+               byteSource = (ByteSource) object;
+            } else if (object instanceof byte[]) {
+               byteSource = ByteSource.wrap((byte[]) object);
+            } else {
+               // This should not happen.
                try {
                   byteSource = ByteSource.wrap(ByteStreams2.toByteArrayAndClose(blob.getPayload().openStream()));
                } catch (IOException e) {