Repository: jclouds Updated Branches: refs/heads/master 90498ae04 -> 3ab335e77
JCLOUDS-1368: Fix off-by-one in slicing algorithm Project: http://git-wip-us.apache.org/repos/asf/jclouds/repo Commit: http://git-wip-us.apache.org/repos/asf/jclouds/commit/eb5db026 Tree: http://git-wip-us.apache.org/repos/asf/jclouds/tree/eb5db026 Diff: http://git-wip-us.apache.org/repos/asf/jclouds/diff/eb5db026 Branch: refs/heads/master Commit: eb5db026da3e205356e7629aec89cd6768073d88 Parents: 90498ae Author: Andrew Gaul <[email protected]> Authored: Thu Feb 8 21:00:33 2018 -0800 Committer: Andrew Gaul <[email protected]> Committed: Thu Feb 8 21:00:33 2018 -0800 ---------------------------------------------------------------------- .../main/java/org/jclouds/blobstore/internal/BaseBlobStore.java | 2 +- .../blobstore/integration/internal/BaseBlobIntegrationTest.java | 2 ++ 2 files changed, 3 insertions(+), 1 deletion(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/jclouds/blob/eb5db026/blobstore/src/main/java/org/jclouds/blobstore/internal/BaseBlobStore.java ---------------------------------------------------------------------- diff --git a/blobstore/src/main/java/org/jclouds/blobstore/internal/BaseBlobStore.java b/blobstore/src/main/java/org/jclouds/blobstore/internal/BaseBlobStore.java index 97d4553..eed9ab1 100644 --- a/blobstore/src/main/java/org/jclouds/blobstore/internal/BaseBlobStore.java +++ b/blobstore/src/main/java/org/jclouds/blobstore/internal/BaseBlobStore.java @@ -360,7 +360,7 @@ public abstract class BaseBlobStore implements BlobStore { long partSize = algorithm.calculateChunkSize(contentLength); int partNumber = 1; // TODO: for InputStream payloads, this buffers all parts in-memory! - while (partNumber < algorithm.getParts()) { + while (partNumber <= algorithm.getParts()) { Payload payload = slicer.slice(blob.getPayload(), algorithm.getCopied(), partSize); BlobUploader b = new BlobUploader(mpu, partNumber++, payload); http://git-wip-us.apache.org/repos/asf/jclouds/blob/eb5db026/blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobIntegrationTest.java ---------------------------------------------------------------------- diff --git a/blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobIntegrationTest.java b/blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobIntegrationTest.java index af713fe..023c870 100644 --- a/blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobIntegrationTest.java +++ b/blobstore/src/test/java/org/jclouds/blobstore/integration/internal/BaseBlobIntegrationTest.java @@ -828,6 +828,8 @@ public class BaseBlobIntegrationTest extends BaseBlobStoreIntegrationTest { assertThat(etag).isNotNull(); Blob blob = blobStore.getBlob(container, blobName); + assertThat(blob.getMetadata().getContentMetadata().getContentLength()).isEqualTo(length); + InputStream is = null; try { is = blob.getPayload().openStream();
